Move convertViewportToWindow and convertWindowToViewport from

content/shell/renderer/layout_test/blink_test_runner.cc

Index: content/shell/renderer/layout_test/blink_test_runner.cc
diff --git a/content/shell/renderer/layout_test/blink_test_runner.cc b/content/shell/renderer/layout_test/blink_test_runner.cc
index 81e77ccc4de6e411003cb6704a17887e81f42a2b..5062d6fc2bd0b6728c7d485f04ccd65958929450 100644
--- a/content/shell/renderer/layout_test/blink_test_runner.cc
+++ b/content/shell/renderer/layout_test/blink_test_runner.cc
@@ -494,7 +494,7 @@ void BlinkTestRunner::SetDeviceScaleFactor(float 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() {