Ignore viewport meta tags when Request Desktop Site is enabled.

third_party/WebKit/Source/core/dom/Document.cpp

Index: third_party/WebKit/Source/core/dom/Document.cpp
diff --git a/third_party/WebKit/Source/core/dom/Document.cpp b/third_party/WebKit/Source/core/dom/Document.cpp
index 3e2e08c5fd83597ce751980686ab01c35092cef5..211ea4405b69fd28fd7bc7aae680c4323f0a09b8 100644
--- a/third_party/WebKit/Source/core/dom/Document.cpp
+++ b/third_party/WebKit/Source/core/dom/Document.cpp
@@ -3134,6 +3134,9 @@ void Document::setViewportDescription(const ViewportDescription& viewportDescrip
     if (viewportDescription == m_viewportDescription)
         return;
 
+    if (settings() && !settings()->viewportMetaEnabled() && viewportDescription.isMetaViewportType())

[johnme, 2016/03/15 11:38:10]

Why do you only disable meta viewport in Request Desktop Mode? Why not also
ignore HandheldFriendlyMeta and MobileOptimizedMeta?

As for @viewport, are you treating that differently because the spec says it
should be implemented the same on mobile and desktop?

[aelias_OOO_until_Jul13, 2016/03/15 19:14:15]

Those are rarely seen, and I think it's especially unlikely that an old site
would send a HandheldFriendlyMeta in response to a desktop UA.  I think there's
an argument to be made that from the point of view of the Blink setting,
!viewportMetaEnabled should disable *all* legacy viewport tags though.
There's already a separate setting "viewportEnabled" for @viewport yes.  Note
that we need to always support it in some form on Android since the magic "980"
value for desktop sites is in an @viewport tag for viewportAndroid.css.

[aelias_OOO_until_Jul13, 2016/03/15 19:29:26]

OK, I made this change.

+        return;
+
     // The UA-defined min-width is used by the processing of legacy meta tags.
     if (!viewportDescription.isSpecifiedByAuthor())
         m_viewportDefaultMinWidth = viewportDescription.minWidth;