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

Side by Side Diff: android_webview/renderer/aw_permission_client.cc

Issue 216983003: Chrome side changes to removing WebFrame parameter from WebPermissionClient, since it's redundant n… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: review comments Created 6 years, 8 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/renderer/aw_permission_client.h" 5 #include "android_webview/renderer/aw_permission_client.h"
6 6
7 #include "content/public/renderer/render_frame.h" 7 #include "content/public/renderer/render_frame.h"
8 #include "third_party/WebKit/public/platform/WebURL.h" 8 #include "third_party/WebKit/public/platform/WebURL.h"
9 #include "third_party/WebKit/public/web/WebFrame.h" 9 #include "third_party/WebKit/public/web/WebFrame.h"
10 #include "url/gurl.h" 10 #include "url/gurl.h"
(...skipping 19 matching lines...) Expand all
30 } 30 }
31 31
32 AwPermissionClient::~AwPermissionClient() { 32 AwPermissionClient::~AwPermissionClient() {
33 } 33 }
34 34
35 bool AwPermissionClient::allowDisplayingInsecureContent( 35 bool AwPermissionClient::allowDisplayingInsecureContent(
36 blink::WebFrame* frame, 36 blink::WebFrame* frame,
37 bool enabled_per_settings, 37 bool enabled_per_settings,
38 const blink::WebSecurityOrigin& origin, 38 const blink::WebSecurityOrigin& origin,
39 const blink::WebURL& url) { 39 const blink::WebURL& url) {
40 return enabled_per_settings ? true : AllowMixedContent(url); 40 return allowDisplayingInsecureContent(enabled_per_settings, origin, url);
41 } 41 }
42 42
43 bool AwPermissionClient::allowRunningInsecureContent( 43 bool AwPermissionClient::allowRunningInsecureContent(
44 blink::WebFrame* frame, 44 blink::WebFrame* frame,
45 bool enabled_per_settings, 45 bool enabled_per_settings,
46 const blink::WebSecurityOrigin& origin, 46 const blink::WebSecurityOrigin& origin,
47 const blink::WebURL& url) { 47 const blink::WebURL& url) {
48 return allowRunningInsecureContent(enabled_per_settings, origin, url);
49 }
50
51 bool AwPermissionClient::allowDisplayingInsecureContent(
52 bool enabled_per_settings,
53 const blink::WebSecurityOrigin& origin,
54 const blink::WebURL& url) {
55 return enabled_per_settings ? true : AllowMixedContent(url);
56 }
57
58 bool AwPermissionClient::allowRunningInsecureContent(
59 bool enabled_per_settings,
60 const blink::WebSecurityOrigin& origin,
61 const blink::WebURL& url) {
48 return enabled_per_settings ? true : AllowMixedContent(url); 62 return enabled_per_settings ? true : AllowMixedContent(url);
49 } 63 }
50 64
51 } // namespace android_webview 65 } // namespace android_webview
OLDNEW
« no previous file with comments | « android_webview/renderer/aw_permission_client.h ('k') | chrome/renderer/content_settings_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698