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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/tool/servers/rebaselineserver.py

Issue 2019923002: Fix pylint unused-* warnings in webkitpy/tool. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 4 years, 7 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: third_party/WebKit/Tools/Scripts/webkitpy/tool/servers/rebaselineserver.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/tool/servers/rebaselineserver.py b/third_party/WebKit/Tools/Scripts/webkitpy/tool/servers/rebaselineserver.py
index e75fe78c7ba67bcdcd5495dca03073deffe90952..45833a3bc8318e246752a3234b298ab6cef9aed4 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/tool/servers/rebaselineserver.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/tool/servers/rebaselineserver.py
@@ -65,7 +65,6 @@ def _rebaseline_test(test_file, baseline_target, baseline_move_to, test_config,
filesystem = test_config.filesystem
scm = test_config.scm
layout_tests_directory = test_config.layout_tests_directory
- results_directory = test_config.results_directory
target_expectations_directory = filesystem.join(
layout_tests_directory, 'platform', baseline_target, test_directory)
test_results_directory = test_config.filesystem.join(
@@ -174,9 +173,7 @@ def get_test_baselines(test_file, test_config):
def platform_from_directory(self, directory):
return self._platforms_by_directory[directory]
- test_path = test_config.filesystem.join(test_config.layout_tests_directory, test_file)
- host = test_config.host
- all_platforms_port = AllPlatformsPort(host)
+ all_platforms_port = AllPlatformsPort(test_config.host)
all_test_baselines = {}
for baseline_extension in ('.txt', '.checksum', '.png'):

Powered by Google App Engine
This is Rietveld 408576698