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

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 4 years 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 bf4e545828ba743c3941d8f95f2ac4eecd5bcef9..798460d17249cf47defae62c944380c1b7e5e6c5 100644
--- a/content/shell/browser/layout_test/blink_test_controller.cc
+++ b/content/shell/browser/layout_test/blink_test_controller.cc
@@ -582,10 +582,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