Move/replace/rename URL-based extension getters from ExtensionService to/in ExtensionSet.

chrome/browser/notifications/desktop_notification_service.cc

Index: chrome/browser/notifications/desktop_notification_service.cc
diff --git a/chrome/browser/notifications/desktop_notification_service.cc b/chrome/browser/notifications/desktop_notification_service.cc
index 8a05b621d62f9d932f4e9c307984af28fd9561df..8c3c8c2d5dad19fde3a7dcd4abcfe7e410f56da2 100644
--- a/chrome/browser/notifications/desktop_notification_service.cc
+++ b/chrome/browser/notifications/desktop_notification_service.cc
@@ -408,7 +408,8 @@ string16 DesktopNotificationService::DisplayNameForOrigin(
   if (origin.SchemeIs(chrome::kExtensionScheme)) {
     ExtensionService* ext_service = profile_->GetExtensionService();
     if (ext_service) {
-      const Extension* extension = ext_service->GetExtensionByURL(origin);
+      const Extension* extension =
+          ext_service->extensions()->GetByURL(ExtensionURLInfo(origin));

[abarth-chromium, 2011/12/07 23:51:44]

Same here.

[Yoyo Zhou, 2011/12/08 20:52:17]

Turns out this one is actually also a WebSecurityOrigin converted into a string
converted into a GURL.

Any thoughts on whether GURL or string16 is better for this? I'd like to be
consistent.

       if (extension)
         return UTF8ToUTF16(extension->name());
     }