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

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

Issue 23441042: Refactor common invalidation framework types (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move DEPS rule Created 7 years, 2 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 6e9816088a76e153df56cb0b1c53e9419038e486..8cc3000b54f431e373ab286e039ac87e02d46454 100644
--- a/chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc
+++ b/chrome/browser/extensions/api/push_messaging/push_messaging_apitest.cc
@@ -80,11 +80,10 @@ class PushMessagingApiTest : public ExtensionApiTest {
void EmitInvalidation(
const invalidation::ObjectId& object_id,
+ int64 version,
const std::string& payload) {
fake_invalidation_service_->EmitInvalidationForTest(
- object_id,
- syncer::Invalidation::kUnknownVersion,
- payload);
+ syncer::Invalidation::Init(object_id, version, payload));
}
PushMessagingAPI* GetAPI() {
@@ -129,11 +128,11 @@ IN_PROC_BROWSER_TEST_F(PushMessagingApiTest, ReceivesPush) {
// each subchannel at install, so trigger the suppressions first.
for (int i = 0; i < 3; ++i) {
EmitInvalidation(
- ExtensionAndSubchannelToObjectId(extension->id(), i), std::string());
+ ExtensionAndSubchannelToObjectId(extension->id(), i), i, std::string());
}
EmitInvalidation(
- ExtensionAndSubchannelToObjectId(extension->id(), 1), "payload");
+ ExtensionAndSubchannelToObjectId(extension->id(), 1), 5, "payload");
EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
}

Powered by Google App Engine
This is Rietveld 408576698