Make test_runner::AppBannerClient an internal detail of components/test_runner.

components/test_runner/test_runner.cc

 // Copyright 2014 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 "components/test_runner/test_runner.h"
 
 #include <stddef.h>
 #include <limits>
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "build/build_config.h"
+#include "components/test_runner/app_banner_client.h"
 #include "components/test_runner/mock_credential_manager_client.h"
 #include "components/test_runner/mock_web_speech_recognizer.h"
 #include "components/test_runner/test_interfaces.h"
 #include "components/test_runner/test_preferences.h"
 #include "components/test_runner/web_content_settings.h"
 #include "components/test_runner/web_test_delegate.h"
 #include "components/test_runner/web_test_proxy.h"
 #include "gin/arguments.h"
 #include "gin/array_buffer.h"
 #include "gin/handle.h"
@@ skipping 1517 matching lines @@
   return runner_->DispatchBeforeInstallPromptEvent(request_id, event_platforms,
                                                    callback);
 }
 
 void TestRunnerBindings::ResolveBeforeInstallPromptPromise(
     int request_id,
     const std::string& platform) {
   if (!runner_)
     return;
 
-  return runner_->ResolveBeforeInstallPromptPromise(request_id, platform);
+  runner_->ResolveBeforeInstallPromptPromise(request_id, platform);
 }
 
 std::string TestRunnerBindings::PlatformName() {
   if (runner_)
     return runner_->platform_name_;
   return std::string();
 }
 
 std::string TestRunnerBindings::TooltipText() {
   if (runner_)
@@ skipping 1362 matching lines @@
 
   delegate_->DispatchBeforeInstallPromptEvent(
       request_id, event_platforms,
       base::Bind(&TestRunner::DispatchBeforeInstallPromptCallback,
                  weak_factory_.GetWeakPtr(), base::Passed(&task)));
 }
 
 void TestRunner::ResolveBeforeInstallPromptPromise(
     int request_id,
     const std::string& platform) {
-  delegate_->ResolveBeforeInstallPromptPromise(request_id, platform);
+  test_interfaces_->GetAppBannerClient()->ResolvePromise(request_id, platform);

[Łukasz Anforowicz, 2016/03/15 21:57:28]

This is what the delegate (i.e. BlinkTestRunner) would have done - let's do it
right here.

 }
 
 void TestRunner::SetPOSIXLocale(const std::string& locale) {
   delegate_->SetLocale(locale);
 }
 
 void TestRunner::SetMIDIAccessorResult(bool result) {
   midi_accessor_result_ = result;
 }
 
@@ skipping 254 matching lines @@
 }
 
 void TestRunner::DidLosePointerLockInternal() {
   bool was_locked = pointer_locked_;
   pointer_locked_ = false;
   if (was_locked)
     web_view_->didLosePointerLock();
 }
 
 }  // namespace test_runner