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

Unified Diff: content/browser/renderer_host/render_widget_host_view_aura_unittest.cc

Issue 1980163002: Use black for resize gutter in tab fullscreen mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2704
Patch Set: Created 4 years, 7 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/renderer_host/render_widget_host_view_aura_unittest.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
index da4428671807336de7bb83b6b9ffba3cc7e72f5f..abf25d5ec67efdd418c33b8112aaba555a5d25ea 100644
--- a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
+++ b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
@@ -152,10 +152,11 @@ class TestOverscrollDelegate : public OverscrollControllerDelegate {
class MockRenderWidgetHostDelegate : public RenderWidgetHostDelegate {
public:
- MockRenderWidgetHostDelegate() : rwh_(nullptr) {}
+ MockRenderWidgetHostDelegate() : rwh_(nullptr), is_fullscreen_(false) {}
~MockRenderWidgetHostDelegate() override {}
const NativeWebKeyboardEvent* last_event() const { return last_event_.get(); }
void set_widget_host(RenderWidgetHostImpl* rwh) { rwh_ = rwh; }
+ void set_is_fullscreen(bool is_fullscreen) { is_fullscreen_ = is_fullscreen; }
protected:
// RenderWidgetHostDelegate:
@@ -172,10 +173,16 @@ class MockRenderWidgetHostDelegate : public RenderWidgetHostDelegate {
if (rwh_)
rwh_->SendScreenRects();
}
+ bool IsFullscreenForCurrentTab(
+ RenderWidgetHostImpl* render_widget_host) const override {
+ return is_fullscreen_;
+ }
private:
scoped_ptr<NativeWebKeyboardEvent> last_event_;
RenderWidgetHostImpl* rwh_;
+ bool is_fullscreen_;
+
DISALLOW_COPY_AND_ASSIGN(MockRenderWidgetHostDelegate);
};
@@ -1717,12 +1724,16 @@ TEST_F(RenderWidgetHostViewAuraTest, DelegatedFrameGutter) {
EXPECT_EQ(gfx::Rect(0, 45, 40, 55), parent_layer->children()[1]->bounds());
EXPECT_EQ(SK_ColorRED, parent_layer->children()[1]->background_color());
+ delegates_.back()->set_is_fullscreen(true);
view_->SetSize(medium_size);
// Right gutter is unnecessary.
ASSERT_EQ(1u, parent_layer->children().size());
EXPECT_EQ(gfx::Rect(0, 45, 40, 50), parent_layer->children()[0]->bounds());
+ // RWH is fullscreen, so gutters should be black.
+ EXPECT_EQ(SK_ColorBLACK, parent_layer->children()[0]->background_color());
+
frame = MakeDelegatedFrame(1.f, medium_size, gfx::Rect(medium_size));
view_->OnSwapCompositorFrame(0, std::move(frame));
EXPECT_EQ(0u, parent_layer->children().size());

Powered by Google App Engine
This is Rietveld 408576698