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

Unified Diff: content/test/test_render_view_host.cc

Issue 1713473002: Make TestRenderWidgetHostView::Show/Hide call through to RWHI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@crash
Patch Set: Update web_contents_impl_unittest.cc Created 4 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 side-by-side diff with in-line comments
Download patch
Index: content/test/test_render_view_host.cc
diff --git a/content/test/test_render_view_host.cc b/content/test/test_render_view_host.cc
index bb5999ecfb06e50eb485fe54f279fea74cc1cc7d..f8dec26992517b32ca17e07de97c028d0ab15029 100644
--- a/content/test/test_render_view_host.cc
+++ b/content/test/test_render_view_host.cc
@@ -54,7 +54,6 @@ void InitNavigateParams(FrameHostMsg_DidCommitProvisionalLoad_Params* params,
TestRenderWidgetHostView::TestRenderWidgetHostView(RenderWidgetHost* rwh)
: rwh_(RenderWidgetHostImpl::From(rwh)),
- is_showing_(false),
is_occluded_(false),
did_swap_compositor_frame_(false) {
rwh_->SetView(this);
@@ -96,16 +95,18 @@ bool TestRenderWidgetHostView::IsSurfaceAvailableForCopy() const {
}
void TestRenderWidgetHostView::Show() {
- is_showing_ = true;
is_occluded_ = false;
+ if (rwh_->is_hidden())
+ rwh_->WasShown(ui::LatencyInfo());
}
void TestRenderWidgetHostView::Hide() {
- is_showing_ = false;
+ if (!rwh_->is_hidden())
+ rwh_->WasHidden();
}
bool TestRenderWidgetHostView::IsShowing() {
- return is_showing_;
+ return !rwh_->is_hidden();
}
void TestRenderWidgetHostView::WasUnOccluded() {

Powered by Google App Engine
This is Rietveld 408576698