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

Unified Diff: Tools/Scripts/webkitpy/common/checkout/baselineoptimizer.py

Issue 421623005: Fix BaselineOptimizerTest break (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Tools/Scripts/webkitpy/common/checkout/baselineoptimizer.py
diff --git a/Tools/Scripts/webkitpy/common/checkout/baselineoptimizer.py b/Tools/Scripts/webkitpy/common/checkout/baselineoptimizer.py
index b709ac857453308c87b873935d4e6edff01434ae..29725c02641ca13ca9f7d98b2d065f395ad626f8 100644
--- a/Tools/Scripts/webkitpy/common/checkout/baselineoptimizer.py
+++ b/Tools/Scripts/webkitpy/common/checkout/baselineoptimizer.py
@@ -314,7 +314,7 @@ class BaselineOptimizer(object):
break
_log.debug("Deleting redundant virtual root expected result.")
- if self._skip_scm_commands:
+ if self._skip_scm_commands and virtual_root_expected_baseline_path in self._files_to_add:
self._files_to_add.remove(virtual_root_expected_baseline_path)
if self._scm.exists(virtual_root_expected_baseline_path):
_log.debug(" Deleting (SCM): " + virtual_root_expected_baseline_path)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698