Ignore viewport meta tags when Request Desktop Site is enabled.

content/browser/frame_host/navigation_controller_impl.cc

Index: content/browser/frame_host/navigation_controller_impl.cc
diff --git a/content/browser/frame_host/navigation_controller_impl.cc b/content/browser/frame_host/navigation_controller_impl.cc
index e24b58e84c6d09a62902022699c380921c95f04c..a00629c09ea8346e17079adb5fd4b4739ff17a37 100644
--- a/content/browser/frame_host/navigation_controller_impl.cc
+++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -1567,6 +1567,10 @@ void NavigationControllerImpl::ClearAllScreenshots() {
   screenshot_manager_->ClearAllScreenshots();
 }
 
+void NavigationControllerImpl::UpdateOverridingUserAgent() {
+  delegate_->UpdateOverridingUserAgent();

[no sievers, 2016/03/15 20:00:05]

Does passing this to the RWH here actually work for every navigation (incl. back
for example)?

+nasko@ also since the NavigationControllerDelegate has comments that it's
temporary.
It seems like overriding WebPreferences would be better through
ContentBrowserClient::OverrideWebkitPrefs(), but it still doesn't seem
straightforward since we associate this override with the navigation entry.

[nasko, 2016/03/18 21:05:25]

Did you meant RenderViewHost? RenderWidgetHost shouldn't be dealing with user
agent information.
How would you know which RenderViewHost to send it to?
 
Yeah, that might be wrong by now :).
The current code in this CL doesn't seem to be associated at all with
NavigationEntry either. Why not take the
ContentBrowserClient::OverrideWebkitPrefs approach and just call
RenderViewHost::OnWebkitPreferencesChanged from the Android code?

[aelias_OOO_until_Jul13, 2016/03/18 22:49:18]

I took that approach initially but it failed DEPS verification when I tried to
upload the patch.  This plumbing is the simplest way I found so far that
appeared valid.

[aelias_OOO_until_Jul13, 2016/04/07 03:06:14]

I was able to remove the unnecessary new NavigationControllerImpl method by
making use of the existing NavigationControllerImpl::delegate().  (The DEPS rule
that I had been violating is that I couldn't call into WebContents from
frame_host/.)

+}
+
 void NavigationControllerImpl::SetSessionStorageNamespace(
     const std::string& partition_id,
     SessionStorageNamespace* session_storage_namespace) {