Add a Create method to DesktopNotificationHandler and stubs for the notification objects.

chrome/browser/notifications/desktop_notification_service.h

Index: chrome/browser/notifications/desktop_notification_service.h
diff --git a/chrome/browser/notifications/desktop_notification_service.h b/chrome/browser/notifications/desktop_notification_service.h
index 50f28f5f5c7ca8d5a043d03b2920a1b2c0c13397..7a9749f6acb48d2ffeae81fc87082767131131db 100644
--- a/chrome/browser/notifications/desktop_notification_service.h
+++ b/chrome/browser/notifications/desktop_notification_service.h
@@ -23,7 +23,9 @@
 #include "googleurl/src/gurl.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebTextDirection.h"
 
+class Extension;
 class Notification;
+class NotificationDelegate;
 class NotificationUIManager;
 class NotificationsPrefsCache;
 class PrefService;
@@ -124,6 +126,20 @@ class DesktopNotificationService : public NotificationObserver,
 
   ContentSetting GetContentSetting(const GURL& origin);
 
+  // Creates a notification balloon for the given origin & content and adds it
+  // to the notification ui manager for display.
+  static void ShowBalloon(GURL origin_url,
+                          GURL icon_url,
+                          const string16& title,
+                          const string16& message,
+                          NotificationDelegate* delegate,
+                          Profile* profile);

[John Gregg, 2011/06/03 00:13:54]

This doesn't make sense to me.  Perhaps this class is over-broadly named, but
the point of it is to connect web pages to the notification UI.

Other places in browser code (which might call your new function), could just
call into the notification UI themselves, basically what the impl of this
function does.  Why doesn't that work?

[Satish, 2011/06/03 07:17:03]

I moved this code here to reduce surface area of calls to the notification code
and stubs required. Do you have any suggestions where else I could move this
method?

[Satish, 2011/06/03 12:29:54]

I have now removed this and rolled back the calling code to what they were
before. It required adding just 2 more mock methods which seems fine.

+
+  // Checks to see if a given origin has permission to create desktop
+  // notifications.  Returns a constant from WebNotificationPresenter
+  // class.
+  static int HasPermission(Profile* profile, const GURL& origin);
+
  private:
   void InitPrefs();
   void StartObserving();