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

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

Issue 958973002: Call WebFrameWidget::setVisibilityState on RFI::WasShown (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_impl.h" 5 #include "content/renderer/render_frame_impl.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 3610 matching lines...) Expand 10 before | Expand all | Expand 10 after
3621 3621
3622 void RenderFrameImpl::WasHidden() { 3622 void RenderFrameImpl::WasHidden() {
3623 FOR_EACH_OBSERVER(RenderFrameObserver, observers_, WasHidden()); 3623 FOR_EACH_OBSERVER(RenderFrameObserver, observers_, WasHidden());
3624 } 3624 }
3625 3625
3626 void RenderFrameImpl::WasShown() { 3626 void RenderFrameImpl::WasShown() {
3627 // TODO(kenrb): Need to figure out how to do this better. Should 3627 // TODO(kenrb): Need to figure out how to do this better. Should
3628 // VisibilityState remain a page-level concept or move to frames? 3628 // VisibilityState remain a page-level concept or move to frames?
3629 // The semantics of 'Show' might have to change here. 3629 // The semantics of 'Show' might have to change here.
3630 if (render_widget_) { 3630 if (render_widget_) {
3631 render_view()->webview()->setVisibilityState( 3631 static_cast<blink::WebFrameWidget*>(render_widget_->webwidget())->
3632 blink::WebPageVisibilityStateVisible, false); 3632 setVisibilityState(blink::WebPageVisibilityStateVisible, false);
3633 } 3633 }
3634 FOR_EACH_OBSERVER(RenderFrameObserver, observers_, WasShown()); 3634 FOR_EACH_OBSERVER(RenderFrameObserver, observers_, WasShown());
3635 } 3635 }
3636 3636
3637 bool RenderFrameImpl::IsHidden() { 3637 bool RenderFrameImpl::IsHidden() {
3638 return GetRenderWidget()->is_hidden(); 3638 return GetRenderWidget()->is_hidden();
3639 } 3639 }
3640 3640
3641 // Tell the embedding application that the URL of the active page has changed. 3641 // Tell the embedding application that the URL of the active page has changed.
3642 void RenderFrameImpl::SendDidCommitProvisionalLoad( 3642 void RenderFrameImpl::SendDidCommitProvisionalLoad(
(...skipping 799 matching lines...) Expand 10 before | Expand all | Expand 10 after
4442 4442
4443 #if defined(ENABLE_BROWSER_CDMS) 4443 #if defined(ENABLE_BROWSER_CDMS)
4444 RendererCdmManager* RenderFrameImpl::GetCdmManager() { 4444 RendererCdmManager* RenderFrameImpl::GetCdmManager() {
4445 if (!cdm_manager_) 4445 if (!cdm_manager_)
4446 cdm_manager_ = new RendererCdmManager(this); 4446 cdm_manager_ = new RendererCdmManager(this);
4447 return cdm_manager_; 4447 return cdm_manager_;
4448 } 4448 }
4449 #endif // defined(ENABLE_BROWSER_CDMS) 4449 #endif // defined(ENABLE_BROWSER_CDMS)
4450 4450
4451 } // namespace content 4451 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698