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

Unified Diff: components/test_runner/test_runner.cc

Issue 2211283004: Drop test-only WebFrameClient params/functions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test fix Created 4 years, 4 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
Index: components/test_runner/test_runner.cc
diff --git a/components/test_runner/test_runner.cc b/components/test_runner/test_runner.cc
index cf41203f1ffe0ada8dd76989f7c8387154cd1e84..0ca306ce54b3d396266ebf9981fbc6bf30d7c1db 100644
--- a/components/test_runner/test_runner.cc
+++ b/components/test_runner/test_runner.cc
@@ -150,7 +150,6 @@ class TestRunnerBindings : public gin::Wrappable<TestRunnerBindings> {
void DumpPermissionClientCallbacks();
void DumpPingLoaderCallbacks();
void DumpResourceLoadCallbacks();
- void DumpResourceRequestPriorities();
void DumpResourceResponseMIMETypes();
void DumpSelectionRect();
void DumpSpellCheckCallbacks();
@@ -400,8 +399,6 @@ gin::ObjectTemplateBuilder TestRunnerBindings::GetObjectTemplateBuilder(
&TestRunnerBindings::DumpPingLoaderCallbacks)
.SetMethod("dumpResourceLoadCallbacks",
&TestRunnerBindings::DumpResourceLoadCallbacks)
- .SetMethod("dumpResourceRequestPriorities",
- &TestRunnerBindings::DumpResourceRequestPriorities)
.SetMethod("dumpResourceResponseMIMETypes",
&TestRunnerBindings::DumpResourceResponseMIMETypes)
.SetMethod("dumpSelectionRect", &TestRunnerBindings::DumpSelectionRect)
@@ -1189,11 +1186,6 @@ void TestRunnerBindings::SetWillSendRequestClearHeader(
runner_->SetWillSendRequestClearHeader(header);
}
-void TestRunnerBindings::DumpResourceRequestPriorities() {
- if (runner_)
- runner_->DumpResourceRequestPriorities();
-}
-
void TestRunnerBindings::SetUseMockTheme(bool use) {
if (runner_)
runner_->SetUseMockTheme(use);
@@ -1888,10 +1880,6 @@ bool TestRunner::policyDelegateShouldNotifyDone() const {
return layout_test_runtime_flags_.policy_delegate_should_notify_done();
}
-bool TestRunner::shouldDumpResourcePriorities() const {
- return layout_test_runtime_flags_.dump_resource_priorities();
-}
-
void TestRunner::setToolTipText(const WebString& text) {
tooltip_text_ = text.utf8();
}
@@ -2542,11 +2530,6 @@ void TestRunner::SetWillSendRequestClearHeader(const std::string& header) {
http_headers_to_clear_.insert(header);
}
-void TestRunner::DumpResourceRequestPriorities() {
- layout_test_runtime_flags_.set_dump_resource_priorities(true);
- OnLayoutTestRuntimeFlagsChanged();
-}
-
void TestRunner::SetUseMockTheme(bool use) {
use_mock_theme_ = use;
blink::setMockThemeEnabledForTest(use);

Powered by Google App Engine
This is Rietveld 408576698