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

Unified Diff: content/browser/frame_host/render_widget_host_view_guest.cc

Issue 2668013005: Turn on AreCrossProcessFramesPossible by default. (Closed)
Patch Set: Fix unit tests, find potential problems. 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/browser/frame_host/render_widget_host_view_guest.cc
diff --git a/content/browser/frame_host/render_widget_host_view_guest.cc b/content/browser/frame_host/render_widget_host_view_guest.cc
index 2b9f1e3ed1be83b5c73eeeaa12f313ad59e12d4d..8dbeb4204cd50fe9f20b14afcc9b2524f2e2c7c8 100644
--- a/content/browser/frame_host/render_widget_host_view_guest.cc
+++ b/content/browser/frame_host/render_widget_host_view_guest.cc
@@ -355,6 +355,7 @@ void RenderWidgetHostViewGuest::UpdateCursor(const WebCursor& cursor) {
// and so we will always hit this code path.
if (!guest_)
return;
+ // TODO(creis): This is wrong. Should check for OOPIF webview mode instead.
alexmos 2017/02/04 02:03:01 I actually think this is ok. The description in t
Charlie Reis 2017/02/13 22:02:49 Acknowledged.
if (SiteIsolationPolicy::AreCrossProcessFramesPossible()) {
RenderWidgetHostViewBase* rwhvb = GetOwnerRenderWidgetHostView();
if (rwhvb)

Powered by Google App Engine
This is Rietveld 408576698