Update viewport_meta setting on any navigation.

content/browser/web_contents/web_contents_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/web_contents/web_contents_impl.h"
 
 #include <stddef.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 2365 matching lines @@
 
 void WebContentsImpl::SetIsVirtualKeyboardRequested(bool requested) {
   virtual_keyboard_requested_ = requested;
 }
 
 bool WebContentsImpl::IsVirtualKeyboardRequested() {
   return virtual_keyboard_requested_;
 }
 
 bool WebContentsImpl::IsOverridingUserAgent() {
-  return GetController().GetVisibleEntry() &&
-         GetController().GetVisibleEntry()->GetIsOverridingUserAgent();
+  return GetController().GetActiveEntry() &&

[nasko, 2017/01/10 17:33:35]

This is not an ok change, since GetActiveEntry is marked deprecated and we'd
like to remove it. I've traced through calls and I think your previous patch is
the more correct one, even though it will potentially notify on navigations
other than back/forward.

[aelias_OOO_until_Jul13, 2017/01/10 21:31:15]

OK then, I reuploaded patch set 1.

+         GetController().GetActiveEntry()->GetIsOverridingUserAgent();
 }
 
 bool WebContentsImpl::IsJavaScriptDialogShowing() const {
   return is_showing_javascript_dialog_;
 }
 
 AccessibilityMode WebContentsImpl::GetAccessibilityMode() const {
   return accessibility_mode_;
 }
 
@@ skipping 3020 matching lines @@
   GetMainFrame()->AddMessageToConsole(
       content::CONSOLE_MESSAGE_LEVEL_WARNING,
       base::StringPrintf("This site does not have a valid SSL "
                          "certificate! Without SSL, your site's and "
                          "visitors' data is vulnerable to theft and "
                          "tampering. Get a valid SSL certificate before"
                          " releasing your website to the public."));
 }
 
 }  // namespace content