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

Unified Diff: Source/web/WebViewImpl.cpp

Issue 382543003: Add a force-layout-height-to-zero layout mode to WebViewImpl (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: make it a setting Created 6 years, 5 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 | « Source/web/WebSettingsImpl.cpp ('k') | Source/web/tests/WebFrameTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/WebViewImpl.cpp
diff --git a/Source/web/WebViewImpl.cpp b/Source/web/WebViewImpl.cpp
index f4e161c00ad670a1bc8f27728e1f3aa15c9dd9c8..cee9430fd72ec8f52fb9d520ee366f2b40f66f49 100644
--- a/Source/web/WebViewImpl.cpp
+++ b/Source/web/WebViewImpl.cpp
@@ -3049,6 +3049,9 @@ void WebViewImpl::updateMainFrameLayoutSize()
}
}
+ if (page()->settings().forceZeroLayoutHeight())
+ layoutSize.height = 0;
abarth-chromium 2014/07/29 16:54:55 The settings machinery looks fine, but I don't und
mkosiba (inactive) 2014/07/29 16:57:43 Thanks! I'll ask Alex to take a look.
+
view->setLayoutSize(layoutSize);
}
« no previous file with comments | « Source/web/WebSettingsImpl.cpp ('k') | Source/web/tests/WebFrameTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698