Move convertViewportToWindow and convertWindowToViewport from

content/shell/renderer/layout_test/blink_test_runner.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/shell/renderer/layout_test/blink_test_runner.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <clocale>
@@ skipping 476 matching lines @@
                                                    bool by_user) {
   Send(new LayoutTestHostMsg_SimulateWebNotificationClose(routing_id(), title,
                                                           by_user));
 }
 
 void BlinkTestRunner::SetDeviceScaleFactor(float factor) {
   content::SetDeviceScaleFactor(render_view(), factor);
 }
 
 float BlinkTestRunner::GetWindowToViewportScale() {
-  return content::GetWindowToViewportScale(render_view());
+  return content::GetWindowToViewportScale(render_view()->GetMainRenderFrame());

[dcheng, 2016/09/02 08:57:39]

And these would also need to be frame-associated as well, right?

[lfg, 2016/09/02 17:36:37]

Yes, I spent a bit of time looking at that, but the callers don't have the
associated frame. For now, I'm just deferring this work to the future, getting
the viewport of the main frame is identical to the old behavior.

[dcheng, 2016/09/08 05:25:47]

Just to confirm: this will explode horribly at some point if we forget to come
back and update this, so we know we need to fix it?

[lfg, 2016/09/08 21:40:43]

Yes, if the main_render_frame_ is null this will cause a nullptr dereference (it
should happen on subframe processes).

 }
 
 void BlinkTestRunner::EnableUseZoomForDSF() {
   base::CommandLine::ForCurrentProcess()->
       AppendSwitch(switches::kEnableUseZoomForDSF);
 }
 
 bool BlinkTestRunner::IsUseZoomForDSFEnabled() {
   return content::IsUseZoomForDSFEnabled();
 }
@@ skipping 509 matching lines @@
 void BlinkTestRunner::ReportLeakDetectionResult(
     const LeakDetectionResult& report) {
   Send(new ShellViewHostMsg_LeakDetectionDone(routing_id(), report));
 }
 
 void BlinkTestRunner::OnDestruct() {
   delete this;
 }
 
 }  // namespace content