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

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>
 #include <cmath>
@@ skipping 593 matching lines @@
              result.substr(strlen("file:////"));
   }
   return RewriteLayoutTestsURL(result).string().utf8();
 }
 
 void BlinkTestRunner::SetLocale(const std::string& locale) {
   setlocale(LC_ALL, locale.c_str());
 }
 
 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 =
       LayoutTestRenderProcessObserver::GetInstance()->test_interfaces();
   interfaces->SetTestIsRunning(false);
   if (interfaces->TestRunner()->ShouldDumpBackForwardList()) {
     SyncNavigationStateVisitor visitor;
     RenderView::ForEach(&visitor);
     Send(new ShellViewHostMsg_CaptureSessionHistory(routing_id()));
   } else {
@@ skipping 408 matching lines @@
   get_bluetooth_events_callbacks_.pop_front();
   callback.Run(events);
 }
 
 void BlinkTestRunner::ReportLeakDetectionResult(
     const LeakDetectionResult& report) {
   Send(new ShellViewHostMsg_LeakDetectionDone(routing_id(), report));
 }
 
 }  // namespace content