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

Unified Diff: Source/web/PageScaleConstraintsSet.cpp

Issue 55073002: [Android WebView] Add a quirk to clobber initial scale in certain cases (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed checking against device-width Created 7 years, 1 month 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 | « no previous file | Source/web/WebSettingsImpl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/PageScaleConstraintsSet.cpp
diff --git a/Source/web/PageScaleConstraintsSet.cpp b/Source/web/PageScaleConstraintsSet.cpp
index 15f6e824d27c58167a67bcd1e952c71d0b2dc37d..b361be9ce18b7a5e6c70349c10fb53d7b7f8dae8 100644
--- a/Source/web/PageScaleConstraintsSet.cpp
+++ b/Source/web/PageScaleConstraintsSet.cpp
@@ -140,7 +140,7 @@ void PageScaleConstraintsSet::adjustForAndroidWebViewQuirks(const ViewportDescri
if (description.zoom == -1) {
if (description.maxWidth.isAuto())
resetInitialScale = true;
- if (useWideViewport || !description.maxWidth.isFixed())
+ if (useWideViewport || description.maxWidth == Length(100, ViewportPercentageWidth))
resetInitialScale = true;
}
if (resetInitialScale)
@@ -156,7 +156,7 @@ void PageScaleConstraintsSet::adjustForAndroidWebViewQuirks(const ViewportDescri
m_pageDefinedConstraints.initialScale *= targetDensityDPIFactor;
m_pageDefinedConstraints.minimumScale *= targetDensityDPIFactor;
m_pageDefinedConstraints.maximumScale *= targetDensityDPIFactor;
- if (wideViewportQuirkEnabled && (!useWideViewport || !description.maxWidth.isFixed()))
+ if (wideViewportQuirkEnabled && (!useWideViewport || description.maxWidth == Length(100, ViewportPercentageWidth)))
adjustedLayoutSizeWidth /= targetDensityDPIFactor;
}
« no previous file with comments | « no previous file | Source/web/WebSettingsImpl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698