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

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: Created 6 years, 3 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 389 matching lines...) Expand 10 before | Expand all | Expand 10 after
400 AwContentBrowserClient::CheckDesktopNotificationPermission( 400 AwContentBrowserClient::CheckDesktopNotificationPermission(
401 const GURL& source_url, 401 const GURL& source_url,
402 content::ResourceContext* context, 402 content::ResourceContext* context,
403 int render_process_id) { 403 int render_process_id) {
404 // Android WebView does not support notifications, so return Denied here. 404 // Android WebView does not support notifications, so return Denied here.
405 return blink::WebNotificationPermissionDenied; 405 return blink::WebNotificationPermissionDenied;
406 } 406 }
407 407
408 void AwContentBrowserClient::ShowDesktopNotification( 408 void AwContentBrowserClient::ShowDesktopNotification(
409 const content::ShowDesktopNotificationHostMsgParams& params, 409 const content::ShowDesktopNotificationHostMsgParams& params,
410 content::RenderFrameHost* render_frame_host, 410 content::BrowserContext* browser_context,
411 scoped_ptr<content::DesktopNotificationDelegate> delegate, 411 scoped_ptr<content::DesktopNotificationDelegate> delegate,
412 base::Closure* cancel_callback) { 412 base::Closure* cancel_callback) {
413 NOTREACHED() << "Android WebView does not support desktop notifications."; 413 NOTREACHED() << "Android WebView does not support desktop notifications.";
414 } 414 }
415 415
416 void AwContentBrowserClient::RequestGeolocationPermission( 416 void AwContentBrowserClient::RequestGeolocationPermission(
417 content::WebContents* web_contents, 417 content::WebContents* web_contents,
418 int bridge_id, 418 int bridge_id,
419 const GURL& requesting_frame, 419 const GURL& requesting_frame,
420 bool user_gesture, 420 bool user_gesture,
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
574 574
575 #if defined(VIDEO_HOLE) 575 #if defined(VIDEO_HOLE)
576 content::ExternalVideoSurfaceContainer* 576 content::ExternalVideoSurfaceContainer*
577 AwContentBrowserClient::OverrideCreateExternalVideoSurfaceContainer( 577 AwContentBrowserClient::OverrideCreateExternalVideoSurfaceContainer(
578 content::WebContents* web_contents) { 578 content::WebContents* web_contents) {
579 return native_factory_->CreateExternalVideoSurfaceContainer(web_contents); 579 return native_factory_->CreateExternalVideoSurfaceContainer(web_contents);
580 } 580 }
581 #endif 581 #endif
582 582
583 } // namespace android_webview 583 } // namespace android_webview
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698