[Android WebView] Add a quirk to clobber initial scale in certain cases

Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 2951 matching lines @@
         return;
 
     ViewportDescription adjustedDescription = description;
     if (settingsImpl()->viewportMetaLayoutSizeQuirk() && adjustedDescription.type == ViewportDescription::ViewportMeta) {
         if (adjustedDescription.maxWidth.type() == ExtendToZoom)
             adjustedDescription.maxWidth = Length(); // auto
         adjustedDescription.minWidth = adjustedDescription.maxWidth;
         adjustedDescription.minHeight = adjustedDescription.maxHeight;
     }
     float oldInitialScale = m_pageScaleConstraintsSet.pageDefinedConstraints().initialScale;
+    float oldUserAgentInitialScale = m_pageScaleConstraintsSet.userAgentConstraints().initialScale;
     m_pageScaleConstraintsSet.updatePageDefinedConstraints(adjustedDescription, m_size);
-    m_pageScaleConstraintsSet.adjustForAndroidWebViewQuirks(adjustedDescription, m_size, page()->settings().layoutFallbackWidth(), deviceScaleFactor(), settingsImpl()->supportDeprecatedTargetDensityDPI(), page()->settings().wideViewportQuirkEnabled(), page()->settings().useWideViewport(), page()->settings().loadWithOverviewMode());
+    m_pageScaleConstraintsSet.adjustForAndroidWebViewQuirks(adjustedDescription, m_size, page()->settings().layoutFallbackWidth(), deviceScaleFactor(), settingsImpl()->supportDeprecatedTargetDensityDPI(), page()->settings().wideViewportQuirkEnabled(), page()->settings().useWideViewport(), page()->settings().loadWithOverviewMode(), settingsImpl()->clobberUserAgentInitialScaleQuirk());
     float newInitialScale = m_pageScaleConstraintsSet.pageDefinedConstraints().initialScale;
     if (oldInitialScale != newInitialScale && newInitialScale != -1)
         m_pageScaleConstraintsSet.setNeedsReset(true);
-
+    if (settingsImpl()->clobberUserAgentInitialScaleQuirk()) {

[aelias_OOO_until_Jul13, 2013/11/01 00:10:07]

Could you put all the logic within this block instead?

[mnaganov (inactive), 2013/11/04 17:20:52]

Done.

+        float newUserAgentInitialScale = m_pageScaleConstraintsSet.userAgentConstraints().initialScale;
+        if (oldUserAgentInitialScale != newUserAgentInitialScale) {
+            m_pageScaleConstraintsSet.setNeedsReset(true);
+            if (mainFrameImpl() && mainFrameImpl()->frameView())
+                mainFrameImpl()->frameView()->setNeedsLayout();
+        }
+    }
 
     updateMainFrameLayoutSize();
 }
 
 void WebViewImpl::updateMainFrameLayoutSize()
 {
     if (m_fixedLayoutSizeLock || !mainFrameImpl())
         return;
 
     FrameView* view = mainFrameImpl()->frameView();
@@ skipping 1142 matching lines @@
     const PageScaleConstraints& constraints = m_pageScaleConstraintsSet.pageDefinedConstraints();
 
     if (!mainFrameImpl() || !mainFrameImpl()->frameView())
         return false;
 
     return mainFrameImpl()->frameView()->layoutSize().width() == m_size.width
         || (constraints.minimumScale == constraints.maximumScale && constraints.minimumScale != -1);
 }
 
 } // namespace WebKit