[Offline Pages] Fix flaky test due to actual loading of database.

chrome/browser/android/offline_pages/offline_page_model_factory.cc

 // Copyright 2015 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/android/offline_pages/offline_page_model_factory.h"
 
 #include <utility>
 
 #include "base/files/file_path.h"
+#include "base/memory/ptr_util.h"
 #include "base/memory/singleton.h"
 #include "base/sequenced_task_runner.h"
 #include "chrome/browser/profiles/incognito_helpers.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_constants.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "components/offline_pages/offline_page_metadata_store_sql.h"
 #include "components/offline_pages/offline_page_model_impl.h"
+#include "components/offline_pages/stub_offline_page_model.h"
 #include "content/public/browser/browser_thread.h"
 
+namespace {
+std::unique_ptr<KeyedService> BuildMockOfflinePageModelFactory(

[fgorski, 2016/09/08 17:18:57]

how about moving this to testing_profile and appropriately sorting out the deps,
per our discussion.

+    content::BrowserContext* context) {
+  return base::MakeUnique<offline_pages::StubOfflinePageModel>();
+}
+}
+
 namespace offline_pages {
 
 OfflinePageModelFactory::OfflinePageModelFactory()
     : BrowserContextKeyedServiceFactory(
           "OfflinePageModel",
           BrowserContextDependencyManager::GetInstance()) {
 }
 
 // static
 OfflinePageModelFactory* OfflinePageModelFactory::GetInstance() {
   return base::Singleton<OfflinePageModelFactory>::get();
 }
 
 // static
 OfflinePageModel* OfflinePageModelFactory::GetForBrowserContext(
     content::BrowserContext* context) {
   return static_cast<OfflinePageModelImpl*>(
       GetInstance()->GetServiceForBrowserContext(context, true));
 }
 
+// static
+BrowserContextKeyedServiceFactory::TestingFactoryFunction
+OfflinePageModelFactory::GetMockFactory() {
+  return &BuildMockOfflinePageModelFactory;
+}
+
 KeyedService* OfflinePageModelFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
   Profile* profile = Profile::FromBrowserContext(context);
   scoped_refptr<base::SequencedTaskRunner> background_task_runner =
       content::BrowserThread::GetBlockingPool()->GetSequencedTaskRunner(
           content::BrowserThread::GetBlockingPool()->GetSequenceToken());
 
   base::FilePath store_path =
       profile->GetPath().Append(chrome::kOfflinePageMetadataDirname);
   std::unique_ptr<OfflinePageMetadataStore> metadata_store(
       new OfflinePageMetadataStoreSQL(background_task_runner, store_path));
 
   base::FilePath archives_dir =
       profile->GetPath().Append(chrome::kOfflinePageArchivesDirname);
 
   return new OfflinePageModelImpl(std::move(metadata_store), archives_dir,
                                   background_task_runner);
 }
 
 }  // namespace offline_pages