Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(30)

Unified Diff: components/test_runner/web_test_proxy.cc

Issue 1839853002: Moving remaining mocks from WebTestProxyBase into TestRunner. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@splitting-web-view-test-client
Patch Set: Rebasing... Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/test_runner/web_test_proxy.h ('k') | components/test_runner/web_test_runner.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/test_runner/web_test_proxy.cc
diff --git a/components/test_runner/web_test_proxy.cc b/components/test_runner/web_test_proxy.cc
index bd5954b7dc8403de4aa242cf0f1a14a199eff52f..16113f9a8144f5ec84ee34bfd87a8e3d77bf574e 100644
--- a/components/test_runner/web_test_proxy.cc
+++ b/components/test_runner/web_test_proxy.cc
@@ -81,8 +81,7 @@ WebTestProxyBase::WebTestProxyBase()
: test_interfaces_(nullptr),
delegate_(nullptr),
web_view_(nullptr),
- web_widget_(nullptr),
- spellcheck_(new SpellCheckClient(this)) {}
+ web_widget_(nullptr) {}
WebTestProxyBase::~WebTestProxyBase() {
test_interfaces_->WindowClosed(this);
@@ -95,11 +94,6 @@ void WebTestProxyBase::SetInterfaces(WebTestInterfaces* interfaces) {
void WebTestProxyBase::SetDelegate(WebTestDelegate* delegate) {
delegate_ = delegate;
- spellcheck_->SetDelegate(delegate);
-}
-
-blink::WebSpellCheckClient* WebTestProxyBase::GetSpellCheckClient() const {
- return spellcheck_.get();
}
std::string WebTestProxyBase::DumpBackForwardLists() {
@@ -141,18 +135,4 @@ void WebTestProxyBase::GetScreenOrientationForTesting(
screen_info.orientationAngle = mock_client->CurrentOrientationAngle();
}
-MockCredentialManagerClient*
-WebTestProxyBase::GetCredentialManagerClientMock() {
- if (!credential_manager_client_.get())
- credential_manager_client_.reset(new MockCredentialManagerClient());
- return credential_manager_client_.get();
-}
-
-void WebTestProxyBase::PostSpellCheckEvent(const blink::WebString& event_name) {
- if (test_interfaces_->GetTestRunner()->shouldDumpSpellCheckCallbacks()) {
- delegate_->PrintMessage(std::string("SpellCheckEvent: ") +
- event_name.utf8().data() + "\n");
- }
-}
-
} // namespace test_runner
« no previous file with comments | « components/test_runner/web_test_proxy.h ('k') | components/test_runner/web_test_runner.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698