Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2968)

Unified Diff: chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc

Issue 327243003: Introduce ProfileInvalidationProvider wrapper for InvalidationService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix clang compilation. Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc
diff --git a/chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc b/chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc
index d97448406ed4d451711d15977a7f6d7eae10868f..79a3a3684614a7106f3c5749ec37d64c9dc97928 100644
--- a/chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc
+++ b/chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc
@@ -11,13 +11,15 @@
#include "chrome/browser/extensions/extension_apitest.h"
#include "chrome/browser/extensions/extension_test_message_listener.h"
#include "chrome/browser/invalidation/fake_invalidation_service.h"
-#include "chrome/browser/invalidation/invalidation_service_factory.h"
+#include "chrome/browser/invalidation/profile_invalidation_provider.h"
+#include "chrome/browser/invalidation/profile_invalidation_provider_factory.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/test/base/ui_test_utils.h"
#include "components/invalidation/fake_invalidator.h"
#include "components/invalidation/invalidation_service.h"
+#include "components/keyed_service/core/keyed_service.h"
#include "google/cacheinvalidation/types.pb.h"
#include "sync/internal_api/public/base/invalidation.h"
#include "testing/gmock/include/gmock/gmock.h"
@@ -26,7 +28,9 @@ using ::testing::SaveArg;
using ::testing::StrictMock;
using ::testing::_;
-using invalidation::InvalidationServiceFactory;
+namespace content {
+class BrowserContext;
+}
namespace extensions {
@@ -39,6 +43,13 @@ invalidation::ObjectId ExtensionAndSubchannelToObjectId(
base::StringPrintf("U/%s/%d", extension_id.c_str(), subchannel_id));
}
+KeyedService* BuildFakeProfileInvalidationProvider(
+ content::BrowserContext* context) {
+ return new invalidation::ProfileInvalidationProvider(
+ scoped_ptr<invalidation::InvalidationService>(
+ new invalidation::FakeInvalidationService));
+}
+
class MockInvalidationMapper : public PushMessagingInvalidationMapper {
public:
MockInvalidationMapper();
@@ -66,8 +77,8 @@ class PushMessagingApiTest : public ExtensionApiTest {
}
virtual void SetUp() OVERRIDE {
- InvalidationServiceFactory::GetInstance()->RegisterTestingFactory(
- invalidation::FakeInvalidationService::Build);
+ invalidation::ProfileInvalidationProviderFactory::GetInstance()->
+ RegisterTestingFactory(BuildFakeProfileInvalidationProvider);
ExtensionApiTest::SetUp();
}
@@ -75,8 +86,10 @@ class PushMessagingApiTest : public ExtensionApiTest {
ExtensionApiTest::SetUpOnMainThread();
fake_invalidation_service_ =
static_cast<invalidation::FakeInvalidationService*>(
- InvalidationServiceFactory::GetInstance()->GetForProfile(
- profile()));
+ static_cast<invalidation::ProfileInvalidationProvider*>(
+ invalidation::ProfileInvalidationProviderFactory::
+ GetInstance()->GetForProfile(profile()))->
+ GetInvalidationService());
}
void EmitInvalidation(

Powered by Google App Engine
This is Rietveld 408576698