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

Unified Diff: content/renderer/render_view_browsertest.cc

Issue 953233002: Add |needs_resize_ack| flag to ViewMsg_Resize (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove DCHECK() 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/public/test/render_widget_test.cc ('k') | content/renderer/render_widget.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_view_browsertest.cc
diff --git a/content/renderer/render_view_browsertest.cc b/content/renderer/render_view_browsertest.cc
index 392256809f2036d16062ec5731d8ef227cd521bb..87e517d44db788111cca26c814e3177179a61baa 100644
--- a/content/renderer/render_view_browsertest.cc
+++ b/content/renderer/render_view_browsertest.cc
@@ -2283,27 +2283,6 @@ TEST_F(RenderViewImplTest, NavigationStartOverride) {
EXPECT_LE(late_nav_reported_start, after_navigation);
}
-class RenderViewImplInitialSizeTest : public RenderViewImplTest {
- public:
- RenderViewImplInitialSizeTest()
- : RenderViewImplTest(), initial_size_(200, 100) {}
-
- protected:
- scoped_ptr<ViewMsg_Resize_Params> InitialSizeParams() override {
- scoped_ptr<ViewMsg_Resize_Params> initial_size_params(
- new ViewMsg_Resize_Params());
- initial_size_params->new_size = initial_size_;
- return initial_size_params.Pass();
- }
-
- gfx::Size initial_size_;
-};
-
-TEST_F(RenderViewImplInitialSizeTest, InitialSize) {
- ASSERT_EQ(initial_size_, view_->GetSize());
- ASSERT_EQ(initial_size_, gfx::Size(view_->GetWebView()->size()));
-}
-
TEST_F(RenderViewImplTest, PreferredSizeZoomed) {
LoadHTML("<body style='margin:0;'><div style='display:inline-block; "
"width:400px; height:400px;'/></body>");
« no previous file with comments | « content/public/test/render_widget_test.cc ('k') | content/renderer/render_widget.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698