Split GetPermissionStatus into GetPermissionStatusForFrame/Worker

content/shell/browser/layout_test/layout_test_notification_manager.cc

Index: content/shell/browser/layout_test/layout_test_notification_manager.cc
diff --git a/content/shell/browser/layout_test/layout_test_notification_manager.cc b/content/shell/browser/layout_test/layout_test_notification_manager.cc
index 791e370072dd3e4accf27933dc9f589cde28eebd..491c9c986f3e0414ce0a781c6389e05c546f8e19 100644
--- a/content/shell/browser/layout_test/layout_test_notification_manager.cc
+++ b/content/shell/browser/layout_test/layout_test_notification_manager.cc
@@ -20,9 +20,7 @@ LayoutTestNotificationManager::CheckPermission(const GURL& origin) {
   return LayoutTestContentBrowserClient::Get()
       ->GetLayoutTestBrowserContext()
       ->GetLayoutTestPermissionManager()
-      ->GetPermissionStatus(PermissionType::NOTIFICATIONS,
-                            origin,
-                            origin);
+      ->GetPermissionStatusForWorker(PermissionType::NOTIFICATIONS, origin);

[benwells, 2017/05/30 06:49:12]

This feels a bit weird - is this always checking for a worker? Or is it more of
a generic call?

[benwells, 2017/05/31 01:48:10]

^^^ ok ... this still feels a bit weird but as it's a test it's ok. Can you add
a comment why the worker version is used?

[raymes, 2017/06/01 02:54:40]

Done.

 }
 
 }  // namespace content