Quit BlinkTestRunner::TestFinished early if main frame is not local.

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 6bc6ad435fec5abb55b280447ed5f99b95955ae4..22648fe862f43acd3284fe379a5aadea8f4b4bb1 100644
--- a/content/shell/renderer/layout_test/blink_test_runner.cc
+++ b/content/shell/renderer/layout_test/blink_test_runner.cc
@@ -611,8 +611,8 @@ void BlinkTestRunner::SetLocale(const std::string& locale) {
 }
 
 void BlinkTestRunner::TestFinished() {
-  if (!is_main_window_) {
-    Send(new ShellViewHostMsg_TestFinishedInSecondaryWindow(routing_id()));
+  if (!is_main_window_ || !render_view()->GetMainRenderFrame()) {

[jochen (gone - plz use gerrit), 2016/03/04 12:38:52]

ah, so you consider all OOPIF of the main window to be part of the main window?

what if you have a test which opens a window in a different domain, and has an
iframe from that different domain in the main window? won't the other window and
the oopif end up in the same process then?

[Łukasz Anforowicz, 2016/03/04 20:41:41]

Correct, except to be precise s/OOPIF/RenderViews/ (i.e. each rendererer
containing a frame from the main window will also have a RenderView +
BlinkTestRunner associated with the window/page-level stuff).
They will be in the same process, but they will have 2 separate RenderViews. 
Creation of both RenderViews will go through
LayoutTestContentRendererClient::WebTestProxyCreated, so they both will have an
associated BlinkTestRunner.  OTOH, only one of these BlinkTestRunners will
receive a SetTestConfiguration (or ReplicateTestConfiguration) message (only the
one associated with the RenderView for the test's main window).  So I think this
is ok?

[Łukasz Anforowicz, 2016/03/04 20:51:19]

i.e. only one of the BlinkTestRunner's will have |is_main_window_| set to true.

+    Send(new ShellViewHostMsg_TestFinishedInSecondaryRenderer(routing_id()));
     return;
   }
   test_runner::WebTestInterfaces* interfaces =