[Push] Register: require user-visible to be present in the manifest

chrome/browser/services/gcm/push_messaging_permission_context.h

Index: chrome/browser/services/gcm/push_messaging_permission_context.h
diff --git a/chrome/browser/services/gcm/push_messaging_permission_context.h b/chrome/browser/services/gcm/push_messaging_permission_context.h
index 2ae4e6cb3281b5a505e399814d5527a6cd7a5ebc..146d20e7329ef1a860a11bc9953a7cc894a2b362 100644
--- a/chrome/browser/services/gcm/push_messaging_permission_context.h
+++ b/chrome/browser/services/gcm/push_messaging_permission_context.h
@@ -9,6 +9,7 @@
 
 #include "components/content_settings/core/common/content_settings_types.h"
 
+class PermissionRequestID;
 class Profile;
 
 namespace gcm {
@@ -34,8 +35,23 @@ class PushMessagingPermissionContext : public PermissionContextBase {
                         const BrowserPermissionCallback& callback) override;
 
  private:
+  FRIEND_TEST_ALL_PREFIXES(PushMessagingPermissionContextTest,
+                           DecidePushPermission);
+
+  // Used to decide the permission for push, once the permission for
+  // Notification has been granted/denied.
+  void DecidePushPermission(const PermissionRequestID& id,
+                            const GURL& requesting_origin,
+                            const GURL& embedding_origin,
+                            const BrowserPermissionCallback& callback,
+                            bool notifications_permission_granted);
+
   Profile* profile_;
 
+  // Must be the last member, to ensure that it will be
+  // destroyed first, which will invalidate weak pointers
+  base::WeakPtrFactory<PushMessagingPermissionContext> weak_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(PushMessagingPermissionContext);
 };