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

Unified Diff: build/android/pylib/flag_changer.py

Issue 22914014: Add more control over sync for Chromium testshell. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed findbugs Created 7 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: build/android/pylib/flag_changer.py
diff --git a/build/android/pylib/flag_changer.py b/build/android/pylib/flag_changer.py
index 6eb9f0b3c675f93feb2a827fc1aa31f5a8e1fb48..35efaf185b872f4fb74e2a7a2844cba34e444f4e 100644
--- a/build/android/pylib/flag_changer.py
+++ b/build/android/pylib/flag_changer.py
@@ -8,9 +8,6 @@ import traceback
import warnings
-# Location where chrome reads command line flags from
-CHROME_COMMAND_FILE = '/data/local/chrome-command-line'
-
class FlagChanger(object):
"""Changes the flags Chrome runs with.
@@ -21,11 +18,14 @@ class FlagChanger(object):
once the tests have completed.
"""
- def __init__(self, android_cmd):
+ def __init__(self, android_cmd,
+ command_line_file=constants.CHROME_COMMAND_LINE_FILE):
self._android_cmd = android_cmd
+ self._command_line_file = command_line_file
+ logging.info('Command line file: %s', self._command_line_file)
# Save the original flags.
- self._orig_line = self._android_cmd.GetFileContents(CHROME_COMMAND_FILE)
+ self._orig_line = self._android_cmd.GetFileContents(self._command_line_file)
if self._orig_line:
self._orig_line = self._orig_line[0].strip()
@@ -90,11 +90,11 @@ class FlagChanger(object):
logging.info('Current flags: %s', self._current_flags)
if self._current_flags:
- self._android_cmd.SetProtectedFileContents(CHROME_COMMAND_FILE,
+ self._android_cmd.SetProtectedFileContents(self._command_line_file,
'chrome ' +
' '.join(self._current_flags))
else:
- self._android_cmd.RunShellCommand('su -c rm ' + CHROME_COMMAND_FILE)
+ self._android_cmd.RunShellCommand('su -c rm ' + self._command_line_file)
def _TokenizeFlags(self, line):
"""Changes the string containing the command line into a list of flags.

Powered by Google App Engine
This is Rietveld 408576698