Split GetPermissionStatus into GetPermissionStatusForFrame/Worker

content/shell/browser/layout_test/layout_test_notification_manager.cc

 // 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.
 
 #include "content/shell/browser/layout_test/layout_test_notification_manager.h"
 
 #include "content/public/browser/permission_type.h"
 #include "content/shell/browser/layout_test/layout_test_browser_context.h"
 #include "content/shell/browser/layout_test/layout_test_content_browser_client.h"
 #include "content/shell/browser/layout_test/layout_test_permission_manager.h"
 
 namespace content {
 
 LayoutTestNotificationManager::LayoutTestNotificationManager() {}
 LayoutTestNotificationManager::~LayoutTestNotificationManager() {}
 
 
 blink::mojom::PermissionStatus
 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