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

Unified Diff: content/shell/browser/layout_test/blink_test_controller.cc

Issue 2566583002: Change allowed bindings to be per RenderFrame instead of per RenderView. (Closed)
Patch Set: Created 3 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: content/shell/browser/layout_test/blink_test_controller.cc
diff --git a/content/shell/browser/layout_test/blink_test_controller.cc b/content/shell/browser/layout_test/blink_test_controller.cc
index e318bf9d1b81e132496a79a3770ba2057b29b142..9273add258b0a2bbb914f6a234510df2a8785f6b 100644
--- a/content/shell/browser/layout_test/blink_test_controller.cc
+++ b/content/shell/browser/layout_test/blink_test_controller.cc
@@ -583,10 +583,9 @@ void BlinkTestController::DiscardMainWindow() {
}
void BlinkTestController::HandleNewRenderFrameHost(RenderFrameHost* frame) {
- // All RenderViewHosts in layout tests should get Mojo bindings.
- RenderViewHost* rvh = frame->GetRenderViewHost();
- if (!(rvh->GetEnabledBindings() & BINDINGS_POLICY_MOJO))
- rvh->AllowBindings(BINDINGS_POLICY_MOJO);
+ // All RenderFrameHosts in layout tests should get Mojo bindings.
+ if (!(frame->GetEnabledBindings() & BINDINGS_POLICY_MOJO))
+ frame->AllowBindings(BINDINGS_POLICY_MOJO);
RenderProcessHost* process = frame->GetProcess();
bool main_window =

Powered by Google App Engine
This is Rietveld 408576698