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

Side by Side Diff: content/browser/web_contents/web_contents_impl.cc

Issue 1814863002: Make page popups work under OOPIF (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: WebContentsImpl::SendScreenRects simplified Created 4 years, 9 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "content/browser/web_contents/web_contents_impl.h" 5 #include "content/browser/web_contents/web_contents_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <cmath> 9 #include <cmath>
10 #include <utility> 10 #include <utility>
(...skipping 2204 matching lines...) Expand 10 before | Expand all | Expand 10 after
2215 delegate_->HideValidationMessage(this); 2215 delegate_->HideValidationMessage(this);
2216 } 2216 }
2217 2217
2218 void WebContentsImpl::OnMoveValidationMessage( 2218 void WebContentsImpl::OnMoveValidationMessage(
2219 const gfx::Rect& anchor_in_root_view) { 2219 const gfx::Rect& anchor_in_root_view) {
2220 if (delegate_) 2220 if (delegate_)
2221 delegate_->MoveValidationMessage(this, anchor_in_root_view); 2221 delegate_->MoveValidationMessage(this, anchor_in_root_view);
2222 } 2222 }
2223 2223
2224 void WebContentsImpl::SendScreenRects() { 2224 void WebContentsImpl::SendScreenRects() {
2225 RenderWidgetHostImpl::From(GetRenderViewHost()->GetWidget()) 2225 for (FrameTreeNode* node : frame_tree_.Nodes()) {
2226 ->SendScreenRects(); 2226 if (node->current_frame_host()->GetRenderWidgetHost())
2227 node->current_frame_host()->GetRenderWidgetHost()->SendScreenRects();
2228 }
2227 2229
2228 RenderWidgetHostViewBase* rwhv = 2230 RenderWidgetHostViewBase* rwhv =
2229 static_cast<RenderWidgetHostViewBase*>(GetRenderWidgetHostView()); 2231 static_cast<RenderWidgetHostViewBase*>(GetRenderWidgetHostView());
2230 if (rwhv) { 2232 if (rwhv) {
2231 SendPageMessage(new PageMsg_UpdateWindowScreenRect( 2233 SendPageMessage(new PageMsg_UpdateWindowScreenRect(
2232 MSG_ROUTING_NONE, rwhv->GetBoundsInRootWindow())); 2234 MSG_ROUTING_NONE, rwhv->GetBoundsInRootWindow()));
2233 } 2235 }
2234 2236
2235 if (browser_plugin_embedder_) 2237 if (browser_plugin_embedder_)
2236 browser_plugin_embedder_->DidSendScreenRects(); 2238 browser_plugin_embedder_->DidSendScreenRects();
(...skipping 2617 matching lines...) Expand 10 before | Expand all | Expand 10 after
4854 else 4856 else
4855 WasHidden(); 4857 WasHidden();
4856 } 4858 }
4857 4859
4858 void WebContentsImpl::SetJavaScriptDialogManagerForTesting( 4860 void WebContentsImpl::SetJavaScriptDialogManagerForTesting(
4859 JavaScriptDialogManager* dialog_manager) { 4861 JavaScriptDialogManager* dialog_manager) {
4860 dialog_manager_ = dialog_manager; 4862 dialog_manager_ = dialog_manager;
4861 } 4863 }
4862 4864
4863 } // namespace content 4865 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698