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

Unified Diff: chrome/browser/services/gcm/push_messaging_browsertest.cc

Issue 822203006: Push API: Replace buggy FutureData test helper with ResultQueue (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@uservisible
Patch Set: Created 5 years, 11 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/services/gcm/push_messaging_browsertest.cc
diff --git a/chrome/browser/services/gcm/push_messaging_browsertest.cc b/chrome/browser/services/gcm/push_messaging_browsertest.cc
index 4f86266a86033dc4fffb1291ce2dec5074a6d6d5..65f6ac0c722870f1f583e87ba04b200c7684664f 100644
--- a/chrome/browser/services/gcm/push_messaging_browsertest.cc
+++ b/chrome/browser/services/gcm/push_messaging_browsertest.cc
@@ -349,11 +349,9 @@ IN_PROC_BROWSER_TEST_F(PushMessagingBrowserTest, PushEventSuccess) {
ASSERT_EQ("true - is controlled", script_result);
GCMClient::IncomingMessage message;
- GCMClient::MessageData messageData;
- messageData.insert(std::pair<std::string, std::string>("data", "testdata"));
- message.data = messageData;
+ message.data["data"] = "testdata";
push_service()->OnMessage(app_id.ToString(), message);
- ASSERT_TRUE(RunScript("pushData.get()", &script_result));
+ ASSERT_TRUE(RunScript("resultQueue.pop()", &script_result));
EXPECT_EQ("testdata", script_result);
}
@@ -385,16 +383,14 @@ IN_PROC_BROWSER_TEST_F(PushMessagingBrowserTest, PushEventNoServiceWorker) {
base::Unretained(&callback)));
GCMClient::IncomingMessage message;
- GCMClient::MessageData messageData;
- messageData.insert(std::pair<std::string, std::string>("data", "testdata"));
- message.data = messageData;
+ message.data["data"] = "testdata";
push_service()->OnMessage(app_id.ToString(), message);
callback.WaitUntilSatisfied();
EXPECT_EQ(app_id.ToString(), callback.app_id());
// No push data should have been received.
- ASSERT_TRUE(RunScript("pushData.getImmediately()", &script_result));
+ ASSERT_TRUE(RunScript("resultQueue.popImmediately()", &script_result));
EXPECT_EQ("null", script_result);
}
@@ -426,16 +422,14 @@ IN_PROC_BROWSER_TEST_F(PushMessagingBrowserTest, PushEventNoPermission) {
base::Unretained(&callback)));
GCMClient::IncomingMessage message;
- GCMClient::MessageData messageData;
- messageData.insert(std::pair<std::string, std::string>("data", "testdata"));
- message.data = messageData;
+ message.data["data"] = "testdata";
push_service()->OnMessage(app_id.ToString(), message);
callback.WaitUntilSatisfied();
EXPECT_EQ(app_id.ToString(), callback.app_id());
// No push data should have been received.
- ASSERT_TRUE(RunScript("pushData.getImmediately()", &script_result));
+ ASSERT_TRUE(RunScript("resultQueue.popImmediately()", &script_result));
EXPECT_EQ("null", script_result);
}
« no previous file with comments | « no previous file | chrome/test/data/push_messaging/push_test.js » ('j') | chrome/test/data/push_messaging/push_test.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698