Adds a context message of the security origin for web notifications.

chrome/browser/notifications/platform_notification_service_impl.h

 // 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.
 
 #ifndef CHROME_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_SERVICE_IMPL_H_
 #define CHROME_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_SERVICE_IMPL_H_
 
 #include "base/gtest_prod_util.h"
 #include "base/memory/singleton.h"
 #include "base/strings/string16.h"
@@ skipping 79 matching lines @@
   // used by tests. Tests are responsible for cleaning up after themselves.
   void SetNotificationUIManagerForTesting(NotificationUIManager* manager);
 
   // Returns a display name for an origin in the process id, to be used in
   // permission infobar or on the frame of the notification toast. Different
   // from the origin itself when dealing with extensions.
   base::string16 DisplayNameForOriginInProcessId(Profile* profile,
                                                  const GURL& origin,
                                                  int process_id) const;
 
+#if defined(ENABLE_EXTENSIONS)
+  bool ExtensionDisplayName(Profile* profile,
+                            const GURL& origin,
+                            int process_id,
+                            base::string16* out) const;
+#endif
+
+  static base::string16 OriginDisplayName(const GURL& origin,

[Peter Beverloo, 2015/01/10 00:03:33]

Muh. I feel like this shouldn't be something in our class, there's a plethora of
features which would benefit from this. (A Push API TODO was added just today.)

An unsuccessful attempt was made in the following CL to add this to
//net/base/net_util.h:
  https://codereview.chromium.org/532523002/

Let me send an e-mail to a few folks for seeing if we can drive that to a
solution. Doesn't have to block this patch - happy to go forth one way or
another, it'll be easy to consolidate the paths.

[dewittj, 2015/01/13 22:18:01]

right, that's the reason this patch has been in limbo.  Best to do it and
consolidate later :)

[Peter Beverloo, 2015/01/14 19:18:14]

Agreed.

+                                          const std::string& languages);
+
   // Weak reference. Ownership maintains with the test.
   NotificationUIManager* notification_ui_manager_for_tests_;
 
   DISALLOW_COPY_AND_ASSIGN(PlatformNotificationServiceImpl);
 };
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_SERVICE_IMPL_H_