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

Issue 11262034: Upstream remaining differences in android/build python files (Closed)

Created:
8 years, 1 month ago by aberent
Modified:
8 years, 1 month ago
Reviewers:
bulach, Anthony Berent
CC:
chromium-reviews, peter+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy+watch_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Upstream remaining differences in android/build python files Re-enable unit tests in single_test_runner (these are no longer flaky) Allow stack trace, since we now admit we are doing an Android version of Chrome Add keep_test_server_ports option to instrumentation tests. BUG=136980 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=164117

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -19 lines) Patch
M build/android/pylib/single_test_runner.py View 3 chunks +13 lines, -15 lines 2 comments Download
M build/android/pylib/test_options_parser.py View 1 chunk +6 lines, -0 lines 0 comments Download
M build/android/run_instrumentation_tests.py View 1 chunk +6 lines, -4 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Anthony Berent
https://codereview.chromium.org/11262034/diff/1/build/android/pylib/single_test_runner.py File build/android/pylib/single_test_runner.py (left): https://codereview.chromium.org/11262034/diff/1/build/android/pylib/single_test_runner.py#oldcode73 build/android/pylib/single_test_runner.py:73: # False) No longer flaky https://codereview.chromium.org/11262034/diff/1/build/android/pylib/single_test_runner.py#oldcode226 build/android/pylib/single_test_runner.py:226: return data ...
8 years, 1 month ago (2012-10-25 16:02:05 UTC) #1
bulach
lgtm, thanks!!
8 years, 1 month ago (2012-10-25 16:39:42 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/aberent@chromium.org/11262034/1
8 years, 1 month ago (2012-10-25 16:45:06 UTC) #3
commit-bot: I haz the power
8 years, 1 month ago (2012-10-25 18:43:27 UTC) #4
Change committed as 164117

Powered by Google App Engine
This is Rietveld 408576698