Skip to content

Commit

Permalink
Revert of [Suggestions] Move ImageManager to the component (patchset …
Browse files Browse the repository at this point in the history
…#6 id:220001 of https://codereview.chromium.org/543753002/)

Reason for revert:
One of the new unit tests failed with uninit memory access: http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20OS%20%28valgrind%29%282%29/builds/43137/

Original issue's description:
> [Suggestions] Move ImageManager to the component
>
> Keeps the fetcher functionality as ImageFetcherImpl in chrome/browser/search/suggestions.
>
> BUG=387751
> TBR=blundell
> TEST=ImageManagerTest,ImageFetcherBrowserTest
>
> Committed: https://crrev.com/5a25f30080dd57d200679546c970d70f2247def9
> Cr-Commit-Position: refs/heads/master@{#296503}

TBR=huangs@chromium.org,blundell@chromium.org,mathp@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=387751

Review URL: https://codereview.chromium.org/600203002

Cr-Commit-Position: refs/heads/master@{#296553}
  • Loading branch information
leizleiz authored and Commit bot committed Sep 24, 2014
1 parent d6a8369 commit da4b14f
Show file tree
Hide file tree
Showing 19 changed files with 623 additions and 758 deletions.
82 changes: 0 additions & 82 deletions chrome/browser/search/suggestions/image_fetcher_impl.cc

This file was deleted.

84 changes: 0 additions & 84 deletions chrome/browser/search/suggestions/image_fetcher_impl.h

This file was deleted.

149 changes: 0 additions & 149 deletions chrome/browser/search/suggestions/image_fetcher_impl_browsertest.cc

This file was deleted.

Loading

0 comments on commit da4b14f

Please sign in to comment.