Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(575)

Side by Side Diff: android_webview/browser/aw_content_browser_client.cc

Issue 578883003: Remove the strict dependency on WebContents for Web Notifications. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: plumb through render_process_id Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "android_webview/browser/aw_content_browser_client.h" 5 #include "android_webview/browser/aw_content_browser_client.h"
6 6
7 #include "android_webview/browser/aw_browser_context.h" 7 #include "android_webview/browser/aw_browser_context.h"
8 #include "android_webview/browser/aw_browser_main_parts.h" 8 #include "android_webview/browser/aw_browser_main_parts.h"
9 #include "android_webview/browser/aw_browser_permission_request_delegate.h" 9 #include "android_webview/browser/aw_browser_permission_request_delegate.h"
10 #include "android_webview/browser/aw_contents_client_bridge_base.h" 10 #include "android_webview/browser/aw_contents_client_bridge_base.h"
(...skipping 378 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 AwContentBrowserClient::CheckDesktopNotificationPermission( 389 AwContentBrowserClient::CheckDesktopNotificationPermission(
390 const GURL& source_url, 390 const GURL& source_url,
391 content::ResourceContext* context, 391 content::ResourceContext* context,
392 int render_process_id) { 392 int render_process_id) {
393 // Android WebView does not support notifications, so return Denied here. 393 // Android WebView does not support notifications, so return Denied here.
394 return blink::WebNotificationPermissionDenied; 394 return blink::WebNotificationPermissionDenied;
395 } 395 }
396 396
397 void AwContentBrowserClient::ShowDesktopNotification( 397 void AwContentBrowserClient::ShowDesktopNotification(
398 const content::ShowDesktopNotificationHostMsgParams& params, 398 const content::ShowDesktopNotificationHostMsgParams& params,
399 content::RenderFrameHost* render_frame_host, 399 content::BrowserContext* browser_context,
400 int render_process_id,
400 scoped_ptr<content::DesktopNotificationDelegate> delegate, 401 scoped_ptr<content::DesktopNotificationDelegate> delegate,
401 base::Closure* cancel_callback) { 402 base::Closure* cancel_callback) {
402 NOTREACHED() << "Android WebView does not support desktop notifications."; 403 NOTREACHED() << "Android WebView does not support desktop notifications.";
403 } 404 }
404 405
405 void AwContentBrowserClient::RequestGeolocationPermission( 406 void AwContentBrowserClient::RequestGeolocationPermission(
406 content::WebContents* web_contents, 407 content::WebContents* web_contents,
407 int bridge_id, 408 int bridge_id,
408 const GURL& requesting_frame, 409 const GURL& requesting_frame,
409 bool user_gesture, 410 bool user_gesture,
(...skipping 165 matching lines...) Expand 10 before | Expand all | Expand 10 after
575 return native_factory_->CreateExternalVideoSurfaceContainer(web_contents); 576 return native_factory_->CreateExternalVideoSurfaceContainer(web_contents);
576 } 577 }
577 #endif 578 #endif
578 579
579 content::DevToolsManagerDelegate* 580 content::DevToolsManagerDelegate*
580 AwContentBrowserClient::GetDevToolsManagerDelegate() { 581 AwContentBrowserClient::GetDevToolsManagerDelegate() {
581 return new AwDevToolsManagerDelegate(); 582 return new AwDevToolsManagerDelegate();
582 } 583 }
583 584
584 } // namespace android_webview 585 } // namespace android_webview
OLDNEW
« no previous file with comments | « android_webview/browser/aw_content_browser_client.h ('k') | chrome/browser/chrome_content_browser_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698