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

Side by Side Diff: content/renderer/render_frame_proxy.cc

Issue 1414663011: Notifying the Out of Process Renderer about Visibility Change of a Remote Frame (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed the Call to WebRemoteFrameImpl::close() and WebViewHelper::reset() in the Destructor of the … Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/renderer/render_frame_proxy.h" 5 #include "content/renderer/render_frame_proxy.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 #include <map> 8 #include <map>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 } 442 }
443 443
444 void RenderFrameProxy::forwardInputEvent(const blink::WebInputEvent* event) { 444 void RenderFrameProxy::forwardInputEvent(const blink::WebInputEvent* event) {
445 Send(new FrameHostMsg_ForwardInputEvent(routing_id_, event)); 445 Send(new FrameHostMsg_ForwardInputEvent(routing_id_, event));
446 } 446 }
447 447
448 void RenderFrameProxy::frameRectsChanged(const blink::WebRect& frame_rect) { 448 void RenderFrameProxy::frameRectsChanged(const blink::WebRect& frame_rect) {
449 Send(new FrameHostMsg_FrameRectChanged(routing_id_, frame_rect)); 449 Send(new FrameHostMsg_FrameRectChanged(routing_id_, frame_rect));
450 } 450 }
451 451
452 void RenderFrameProxy::visibilityChanged(bool visible) {
453 Send(new FrameHostMsg_VisibilityChanged(routing_id_, visible));
454 }
455
452 void RenderFrameProxy::didChangeOpener(blink::WebFrame* opener) { 456 void RenderFrameProxy::didChangeOpener(blink::WebFrame* opener) {
453 // A proxy shouldn't normally be disowning its opener. It is possible to get 457 // A proxy shouldn't normally be disowning its opener. It is possible to get
454 // here when a proxy that is being detached clears its opener, in which case 458 // here when a proxy that is being detached clears its opener, in which case
455 // there is no need to notify the browser process. 459 // there is no need to notify the browser process.
456 if (!opener) 460 if (!opener)
457 return; 461 return;
458 462
459 // Only a LocalFrame (i.e., the caller of window.open) should be able to 463 // Only a LocalFrame (i.e., the caller of window.open) should be able to
460 // update another frame's opener. 464 // update another frame's opener.
461 DCHECK(opener->isWebLocalFrame()); 465 DCHECK(opener->isWebLocalFrame());
462 466
463 int opener_routing_id = 467 int opener_routing_id =
464 RenderFrameImpl::FromWebFrame(opener->toWebLocalFrame())->GetRoutingID(); 468 RenderFrameImpl::FromWebFrame(opener->toWebLocalFrame())->GetRoutingID();
465 Send(new FrameHostMsg_DidChangeOpener(routing_id_, opener_routing_id)); 469 Send(new FrameHostMsg_DidChangeOpener(routing_id_, opener_routing_id));
466 } 470 }
467 471
468 void RenderFrameProxy::advanceFocus(blink::WebFocusType type, 472 void RenderFrameProxy::advanceFocus(blink::WebFocusType type,
469 blink::WebLocalFrame* source) { 473 blink::WebLocalFrame* source) {
470 int source_routing_id = RenderFrameImpl::FromWebFrame(source)->GetRoutingID(); 474 int source_routing_id = RenderFrameImpl::FromWebFrame(source)->GetRoutingID();
471 Send(new FrameHostMsg_AdvanceFocus(routing_id_, type, source_routing_id)); 475 Send(new FrameHostMsg_AdvanceFocus(routing_id_, type, source_routing_id));
472 } 476 }
473 477
474 void RenderFrameProxy::frameFocused() { 478 void RenderFrameProxy::frameFocused() {
475 Send(new FrameHostMsg_FrameFocused(routing_id_)); 479 Send(new FrameHostMsg_FrameFocused(routing_id_));
476 } 480 }
477 481
478 } // namespace 482 } // namespace
OLDNEW
« no previous file with comments | « content/renderer/render_frame_proxy.h ('k') | third_party/WebKit/Source/core/frame/RemoteFrame.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698