[Suggestions] Create ImageEncoder, to abstract away image encode/decode.

chrome/browser/search/suggestions/suggestions_service_factory.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/search/suggestions/suggestions_service_factory.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/profiles/incognito_helpers.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search/suggestions/image_fetcher_impl.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "components/leveldb_proto/proto_database.h"
 #include "components/leveldb_proto/proto_database_impl.h"
 #include "components/pref_registry/pref_registry_syncable.h"
 #include "components/suggestions/blacklist_store.h"
+#include "components/suggestions/image_encoder.h"
 #include "components/suggestions/image_fetcher.h"
 #include "components/suggestions/image_manager.h"
+#include "components/suggestions/jpeg_image_encoder.h"
 #include "components/suggestions/proto/suggestions.pb.h"
 #include "components/suggestions/suggestions_service.h"
 #include "components/suggestions/suggestions_store.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 
 using content::BrowserThread;
 
 namespace suggestions {
 
@@ skipping 36 matching lines @@
       new SuggestionsStore(the_profile->GetPrefs()));
   scoped_ptr<BlacklistStore> blacklist_store(
       new BlacklistStore(the_profile->GetPrefs()));
 
   scoped_ptr<leveldb_proto::ProtoDatabaseImpl<ImageData> > db(
       new leveldb_proto::ProtoDatabaseImpl<ImageData>(background_task_runner));
 
   base::FilePath database_dir(
       the_profile->GetPath().Append(FILE_PATH_LITERAL("Thumbnails")));
 
+  scoped_ptr<JpegImageEncoder> image_encoder(new JpegImageEncoder());
   scoped_ptr<ImageFetcherImpl> image_fetcher(
       new ImageFetcherImpl(the_profile->GetRequestContext()));
   scoped_ptr<ImageManager> thumbnail_manager(new ImageManager(
       image_fetcher.PassAs<ImageFetcher>(),
+      image_encoder.PassAs<ImageEncoder>(),

[blundell, 2014/10/07 14:13:59]

hmm, I think there was a chromium-dev@ thread about not needing the PassAs()
anymore, mumblemumblemumble C++11

[Mathieu, 2014/10/07 16:00:29]

Done.

       db.PassAs<leveldb_proto::ProtoDatabase<ImageData> >(), database_dir));
   return new SuggestionsService(
       the_profile->GetRequestContext(), suggestions_store.Pass(),
       thumbnail_manager.Pass(), blacklist_store.Pass());
 }
 
 void SuggestionsServiceFactory::RegisterProfilePrefs(
     user_prefs::PrefRegistrySyncable* registry) {
   SuggestionsService::RegisterProfilePrefs(registry);
 }
 
 }  // namespace suggestions