DescriptionRevert of Add a force-layout-height-to-zero layout mode to WebViewImpl (https://codereview.chromium.org/382543003/)
Reason for revert:
Breaks WebKit Android (Nexus4)
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/17772
C 75.423s Main [FAIL] WebFrameTest.SetForceZeroLayoutHeight:
C 75.423s Main ../../third_party/WebKit/Source/web/tests/WebFrameTest.cpp:1396: Failure
C 75.423s Main Value of: webViewHelper.webViewImpl()->mainFrameImpl()->frameView()->layoutSize().height()
C 75.423s Main Actual: 735
C 75.423s Main Expected: viewportHeight
C 75.423s Main Which is: 480
Original issue's description:
> Add a force-layout-height-to-zero layout mode to WebViewImpl
>
> This makes it possible for the android_webview to implement
> 'wrap_content' mode for the view's height without completely ignoring
> the viewport meta tag (like it's doing now).
>
> BUG=392460
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179705
TBR=mkosiba@chromium.org
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179713
Patch Set 1 #
Created: 6 years, 4 months ago
(Patch set is too large to download)
Messages
Total messages: 8 (0 generated)
|