[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

Add --gtest_also_run_disabled_tests in AddJavaTestOptions for test_runner.py. Add the run_disabled logic in instrumentation_test_instance.py.

BUG=669632
Committed: https://crrev.com/7cd65ac511114dc5ee3ad8d392712ef66a76fdde
Cr-Commit-Position: refs/heads/master@{#435842}

[shenghuazhang, 2016-12-02 02:16:32]

Description was changed from

==========
[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

BUG=669632
==========

to

==========
[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

Add --gtest_also_run_disabled_tests in AddJavaTestOptions for test_runner.py.
Add the run_disabled logic in instrumentation_test_instance.py.

BUG=669632
==========

[shenghuazhang, 2016-12-02 02:16:32]

shenghuazhang@chromium.org changed reviewers:
+ jbudorick@chromium.org

[jbudorick, 2016-12-02 02:19:35]

https://codereview.chromium.org/2544533006/diff/1/build/android/pylib/instrum...
File build/android/pylib/instrumentation/instrumentation_test_instance.py
(right):

https://codereview.chromium.org/2544533006/diff/1/build/android/pylib/instrum...
build/android/pylib/instrumentation/instrumentation_test_instance.py:556: if not
self._run_disabled:
Without the property, we should be able to use args.run_disabled directly here.

https://codereview.chromium.org/2544533006/diff/1/build/android/pylib/instrum...
build/android/pylib/instrumentation/instrumentation_test_instance.py:633: def
gtest_also_run_disabled_tests(self):
If this isn't used, don't add it. We can add it when we need it.

[shenghuazhang, 2016-12-02 02:25:40]

Removed property '_run_disabled'.

https://codereview.chromium.org/2544533006/diff/1/build/android/pylib/instrum...
File build/android/pylib/instrumentation/instrumentation_test_instance.py
(right):

https://codereview.chromium.org/2544533006/diff/1/build/android/pylib/instrum...
build/android/pylib/instrumentation/instrumentation_test_instance.py:633: def
gtest_also_run_disabled_tests(self):
On 2016/12/02 02:19:35, jbudorick wrote:
> If this isn't used, don't add it. We can add it when we need it.

Done.

[jbudorick, 2016-12-02 02:26:21]

lgtm

[shenghuazhang, 2016-12-02 02:27:33]

The CQ bit was checked by shenghuazhang@chromium.org

[commit-bot: I haz the power, 2016-12-02 02:28:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-02 03:30:11]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480645653730120,
"parent_rev": "46fcfe68e03e2d9d4ca8cfd13bfeb47981750eeb", "commit_rev":
"3ddde1a3556d94e4d42325845aa1a70c4a9e26fe"}

[commit-bot: I haz the power, 2016-12-02 03:30:41]

Description was changed from

==========
[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

Add --gtest_also_run_disabled_tests in AddJavaTestOptions for test_runner.py.
Add the run_disabled logic in instrumentation_test_instance.py.

BUG=669632
==========

to

==========
[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

Add --gtest_also_run_disabled_tests in AddJavaTestOptions for test_runner.py.
Add the run_disabled logic in instrumentation_test_instance.py.

BUG=669632
==========

[commit-bot: I haz the power, 2016-12-02 03:30:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-02 03:32:29]

Description was changed from

==========
[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

Add --gtest_also_run_disabled_tests in AddJavaTestOptions for test_runner.py.
Add the run_disabled logic in instrumentation_test_instance.py.

BUG=669632
==========

to

==========
[Android] Add '--gtest_also_run_disabled_tests' for instrumentation tests

Add --gtest_also_run_disabled_tests in AddJavaTestOptions for test_runner.py.
Add the run_disabled logic in instrumentation_test_instance.py.

BUG=669632

Committed: https://crrev.com/7cd65ac511114dc5ee3ad8d392712ef66a76fdde
Cr-Commit-Position: refs/heads/master@{#435842}
==========

[commit-bot: I haz the power, 2016-12-02 03:32:30]

Patchset 2 (id:??) landed as
https://crrev.com/7cd65ac511114dc5ee3ad8d392712ef66a76fdde
Cr-Commit-Position: refs/heads/master@{#435842}