Add NotifyOnShowSettings implementation of notification provider API

chrome/test/data/extensions/api_test/notification_provider/basic_usage/background.js

 // Copyright 2014 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.
 
 const notificationProvider = chrome.notificationProvider;
 
+function createNotification(notificationId, options) {
+  return new Promise(function (resolve, reject) {
+    chrome.notifications.create(notificationId, options, function (id) {
+      if (chrome.runtime.lastError) {
+        reject(new Error("Unable to create notification"));
+        return;
+      }
+      resolve(id);
+      return;
+    });
+  });
+};
+
 function notifyOnCleared(senderId, notificationId) {
   return new Promise(function (resolve, reject) {
     notificationProvider.notifyOnCleared(senderId,
                                          notificationId,
                                          function (wasCleared) {
       if (chrome.runtime.lastError || !wasCleared) {
         reject(new Error("Unable to send onClear message"));
         return;
       }
       resolve(wasCleared);
@@ skipping 60 matching lines @@
       resolve(notifierExists);
       return;
     });
   });
 };
 
 function failTest(testName) {
   chrome.test.fail(testName);
 };
 
+function listenForShowSettings() {}
+
 function testFunctions() {
-  var testName = "testFunctions";
-  var notifierId;
-  var notId;
-  notificationProvider.onCreated.addListener(function(senderId,
-                                                      notificationId,
-                                                      content) {
-    notifierId = senderId;
-    notId = notificationId;
+  var myId = chrome.runtime.id;
+  var id1 = "id1";
+  var content = {
+    type: "basic",
+    iconUrl: "icon.png",
+    title: "Title",
+    message: "This is the message."
+  };
 
-    notifyOnClicked(notifierId, notId)
-      .catch(function() { failTest("NotifyOnClickedFailed"); })
-      .then(function() { return notifyOnButtonClicked(notifierId, notId, 0); })
-      .catch(function() { failTest("NotifyOnButtonClickedFailed"); })
-      .then(function () { return notifyOnCleared(notifierId, notId); })
-      .catch(function() { failTest("NotifyOnClearedFailed"); })
-      .then(function () { return notifyOnPermissionLevelChanged(notifierId,
-                                                                "granted"); })
-      .catch(function() { failTest("NotifyOnPermissionLevelChangedFailed"); })
-      .then(function () { return notifyOnShowSettings(notifierId); })
-      .catch(function() { failTest("NotifyOnShowSettingsFailed"); })
-      .then(function() { chrome.test.succeed(testName); });
-  });
+  // Add listener to onShowSettings to show that this extension has advanced
+  // settings.
+  chrome.notifications.onShowSettings.addListener(listenForShowSettings);
+
+  createNotification(id1, content)

[Pete Williamson, 2014/08/11 17:53:50]

You probably don't need all these catch statements.  Each catch will catch
anything in the promise chain that failed before it.  So, instead of alternating
then and catch, just do as many thens as you want, and then catch.  You might
not even catch until the end unless there was some resource that needed to be
cleaned up before you proceeded.

[liyanhou, 2014/08/11 18:24:06]

I was doing that for more detailed error messages to show which function fails
in case of failure. Do you think it would be better to use less catches? 

[Pete Williamson, 2014/08/11 18:31:49]

OK, in that case these are fine.  I just wanted to see if this was intended.

+    .catch(function() { failTest("notifications.create"); })
+    .then(function() { return notifyOnClicked(myId, id1); })
+    .catch(function() { failTest("NotifyOnClickedFailed"); })
+    .then(function() { return notifyOnButtonClicked(myId, id1, 0); })
+    .catch(function() { failTest("NotifyOnButtonClickedFailed"); })
+    .then(function () { return notifyOnCleared(myId, id1); })
+    .catch(function() { failTest("NotifyOnClearedFailed"); })
+    // Notify the sender that the permission level is changed by the user
+    .then(function () { return notifyOnPermissionLevelChanged(myId,
+                                                              "granted"); })
+    .catch(function() { failTest("NotifyOnPermissionLevelChangedFailed"); })
+    .then(function () { return notifyOnShowSettings(myId); })
+    .catch(function() { failTest("NotifyOnShowSettingsFailed1"); })
+    // Try to notify a non existent sender that a user checked its settings
+    .then(function () { return notifyOnShowSettings("DoesNotExist"); })
+    // Fail if it returns true
+    .then(function() { failTest("NotifyOnShowSettingsFailed2"); })
+    .catch(function() { chrome.test.succeed(); })
 };
 
 chrome.test.runTests([ testFunctions ]);