Make PageScaleFactor work for oopif subframes.

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 740 matching lines @@
                         OnDidDisplayContentWithCertificateErrors)
     IPC_MESSAGE_HANDLER(FrameHostMsg_DidRunContentWithCertificateErrors,
                         OnDidRunContentWithCertificateErrors)
     IPC_MESSAGE_HANDLER(FrameHostMsg_RegisterProtocolHandler,
                         OnRegisterProtocolHandler)
     IPC_MESSAGE_HANDLER(FrameHostMsg_UnregisterProtocolHandler,
                         OnUnregisterProtocolHandler)
     IPC_MESSAGE_HANDLER(FrameHostMsg_UpdatePageImportanceSignals,
                         OnUpdatePageImportanceSignals)
     IPC_MESSAGE_HANDLER(FrameHostMsg_Find_Reply, OnFindReply)
+    IPC_MESSAGE_HANDLER(FrameHostMsg_RestorePageScaleFactorOnLoad,
+                        SetPageScaleFactorForSubframes)

[kenrb, 2017/01/24 17:13:27]

This message should only ever come from the top-level frame, right?

Could we move the handler to RenderFrame, which can verify that it is the
correct frame to be receiving the message, and have it call the delegate method?

[wjmaclean, 2017/01/24 18:24:26]

Sure, I'll look into doing that and include it in the next CL.

[wjmaclean, 2017/01/24 21:48:47]

Done.

 #if BUILDFLAG(ENABLE_PLUGINS)
     IPC_MESSAGE_HANDLER(FrameHostMsg_PepperInstanceCreated,
                         OnPepperInstanceCreated)
     IPC_MESSAGE_HANDLER(FrameHostMsg_PepperInstanceDeleted,
                         OnPepperInstanceDeleted)
     IPC_MESSAGE_HANDLER(FrameHostMsg_PepperPluginHung, OnPepperPluginHung)
     IPC_MESSAGE_HANDLER(FrameHostMsg_PepperStartsPlayback,
                         OnPepperStartsPlayback)
     IPC_MESSAGE_HANDLER(FrameHostMsg_PepperStopsPlayback,
                         OnPepperStopsPlayback)
@@ skipping 237 matching lines @@
   DCHECK(result.second);
   return base::Bind(&WebContentsImpl::RemoveBindingSet,
                     weak_factory_.GetWeakPtr(), interface_name);
 }
 
 void WebContentsImpl::UpdateDeviceScaleFactor(double device_scale_factor) {
   SendPageMessage(
       new PageMsg_SetDeviceScaleFactor(MSG_ROUTING_NONE, device_scale_factor));
 }
 
+void WebContentsImpl::SetPageScaleFactorForSubframes(double page_scale_factor) {
+  SendPageMessage(
+      new PageMsg_SetPageScaleFactor(MSG_ROUTING_NONE, page_scale_factor));
+}
+
 void WebContentsImpl::GetScreenInfo(ScreenInfo* screen_info) {
   if (GetView())
     GetView()->GetScreenInfo(screen_info);
 }
 
 std::unique_ptr<WebUI> WebContentsImpl::CreateSubframeWebUI(
     const GURL& url,
     const std::string& frame_name) {
   DCHECK(!frame_name.empty());
   return CreateWebUI(url, frame_name);
@@ skipping 4391 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