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

Unified Diff: chrome/renderer/content_settings_observer.cc

Issue 138333006: Now that RenderFrame is 1:1 with WebFrame, add the associated getters and start cleaning up assumpt… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/content_settings_observer.cc
===================================================================
--- chrome/renderer/content_settings_observer.cc (revision 244810)
+++ chrome/renderer/content_settings_observer.cc (working copy)
@@ -155,6 +155,7 @@
is_interstitial_page_(false),
npapi_plugins_blocked_(false) {
ClearBlockedContentSettings();
+ render_frame->GetWebFrame()->setPermissionClient(this);
}
ContentSettingsObserver::~ContentSettingsObserver() {
@@ -183,10 +184,6 @@
}
}
-void ContentSettingsObserver::WebFrameCreated(blink::WebFrame* frame) {
- frame->setPermissionClient(this);
-}
-
bool ContentSettingsObserver::OnMessageReceived(const IPC::Message& message) {
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(ContentSettingsObserver, message)

Powered by Google App Engine
This is Rietveld 408576698