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

Unified Diff: chrome/test/data/extensions/api_test/gcm/functions/register_without_key/register.js

Issue 167063004: [GCM] Move the manifest key checking logic to gcm.register function (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address feedback Created 6 years, 10 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/test/data/extensions/api_test/gcm/functions/register_without_key/register.js
diff --git a/chrome/test/data/extensions/api_test/gcm/functions/register/register.js b/chrome/test/data/extensions/api_test/gcm/functions/register_without_key/register.js
similarity index 66%
copy from chrome/test/data/extensions/api_test/gcm/functions/register/register.js
copy to chrome/test/data/extensions/api_test/gcm/functions/register_without_key/register.js
index e1a20e20d1507eacfda9de53dd07e4e3414b3794..66255eb763af6b61dc5f173662b1892b5b400091 100644
--- a/chrome/test/data/extensions/api_test/gcm/functions/register/register.js
+++ b/chrome/test/data/extensions/api_test/gcm/functions/register_without_key/register.js
@@ -6,7 +6,10 @@ chrome.test.runTests([
function testRegister() {
var senderIds = ["Sender1", "Sender2"];
chrome.gcm.register(senderIds, function(registrationId) {
- chrome.test.succeed();
+ if (chrome.runtime.lastError.message == "Manifest key was missing.")
+ chrome.test.succeed();
+ else
+ chrome.test.fail("gcm.register should fail.");
});
}
]);

Powered by Google App Engine
This is Rietveld 408576698