Group the different permission related methods in the content api.

content/shell/browser/layout_test/layout_test_content_browser_client.cc

Index: content/shell/browser/layout_test/layout_test_content_browser_client.cc
diff --git a/content/shell/browser/layout_test/layout_test_content_browser_client.cc b/content/shell/browser/layout_test/layout_test_content_browser_client.cc
index c6b033f612a9738b20416ec2ab799afe4e6f61a6..47228c7e8d3e3914c41545062a51a3f3e5ccf386 100644
--- a/content/shell/browser/layout_test/layout_test_content_browser_client.cc
+++ b/content/shell/browser/layout_test/layout_test_content_browser_client.cc
@@ -12,18 +12,6 @@ namespace {
 
 LayoutTestContentBrowserClient* g_layout_test_browser_client;
 
-void RequestDesktopNotificationPermissionOnIO(
-    const GURL& source_origin,
-    RenderFrameHost* render_frame_host,
-    const base::Callback<void(blink::WebNotificationPermission)>& callback) {
-  LayoutTestNotificationManager* manager =
-      LayoutTestContentBrowserClient::Get()->GetLayoutTestNotificationManager();
-  if (manager)
-    manager->RequestPermission(source_origin, callback);

[Peter Beverloo, 2014/10/20 17:32:16]

Removing this is not OK and will break tests.

In order to properly test Web Exposed behavior, layout tests need a way to
control whether permission is granted or not. This code is part of that (there
is more, but c/s/b/ is being refactored quite heavily right now). It defers the
RequestPermission call to a "fake" LayoutTestNotificationManager instead.

[Miguel Garcia, 2014/10/21 17:17:12]

Fixed now

On 2014/10/20 17:32:16, Peter Beverloo wrote:

-  else
-    callback.Run(blink::WebNotificationPermissionAllowed);
-}
-
 }  // namespace
 
 LayoutTestContentBrowserClient::LayoutTestContentBrowserClient() {
@@ -50,19 +38,6 @@ LayoutTestContentBrowserClient::GetLayoutTestNotificationManager() {
   return layout_test_notification_manager_.get();
 }
 
-void LayoutTestContentBrowserClient::RequestDesktopNotificationPermission(
-    const GURL& source_origin,
-    RenderFrameHost* render_frame_host,
-    const base::Callback<void(blink::WebNotificationPermission)>& callback) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  BrowserThread::PostTask(BrowserThread::IO,
-                          FROM_HERE,
-                          base::Bind(&RequestDesktopNotificationPermissionOnIO,
-                                     source_origin,
-                                     render_frame_host,
-                                     callback));
-}
-
 blink::WebNotificationPermission
 LayoutTestContentBrowserClient::CheckDesktopNotificationPermission(
     const GURL& source_url,