Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

BUG=

Committed: https://crrev.com/c4f7db8464349b98b0c75f098b73b9eeaa863bf0
Cr-Commit-Position: refs/heads/master@{#387937}

[Yoland Yan(Google), 2016-04-05 02:52:24]

yolandyan@google.com changed reviewers:
+ jbudorick@chromium.org

[jbudorick, 2016-04-05 13:22:25]

Where is @FlakyTest...?

Also, when you do this conversion, please move any relevant comments into the
message.

https://codereview.chromium.org/1856273002/diff/1/chrome/android/javatests/sr...
File chrome/android/javatests/src/org/chromium/chrome/browser/AudioTest.java
(right):

https://codereview.chromium.org/1856273002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/AudioTest.java:49:
@FlakyTest
e.g., add a message about crbug.com/331122 here

[Yoland Yan(Google), 2016-04-06 18:17:19]

yolandyan@google.com changed reviewers:
+ mikecase@chromium.org

[Yoland Yan(Google), 2016-04-06 18:17:20]

https://codereview.chromium.org/1856273002/diff/1/chrome/android/javatests/sr...
File chrome/android/javatests/src/org/chromium/chrome/browser/AudioTest.java
(right):

https://codereview.chromium.org/1856273002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/AudioTest.java:49:
@FlakyTest
On 2016/04/05 13:22:25, jbudorick wrote:
> e.g., add a message about crbug.com/331122 here

Done.

[mikecase (-- gone --), 2016-04-07 18:03:46]

non-owner lgtm

[jbudorick, 2016-04-07 18:11:03]

also non-owner lgtm

[chromium-reviews, 2016-04-07 18:14:20]

Damn

On Thu, Apr 7, 2016, 11:11 <jbudorick@chromium.org> wrote:

> also non-owner lgtm
>
> https://codereview.chromium.org/1856273002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Yoland Yan(Google), 2016-04-08 00:08:21]

yolandyan@google.com changed reviewers:
+ dfalcantara@chromium.org, feldman@chromium.org, pfeldman@chromium.org,
sievers@chromium.org, xunjieli@chromium.org

[Yoland Yan(Google), 2016-04-08 00:08:22]

For detail about the change, please check this thread:
https://groups.google.com/a/google.com/forum/#!topic/clank-team/jG_A_kAhilo

[gone, 2016-04-08 00:16:18]

lgtm

https://chromiumcodereview.appspot.com/1856273002/diff/40001/chrome/android/j...
File
chrome/android/javatests/src/org/chromium/chrome/browser/TabsOpenedFromExternalAppTest.java
(right):

https://chromiumcodereview.appspot.com/1856273002/diff/40001/chrome/android/j...
chrome/android/javatests/src/org/chromium/chrome/browser/TabsOpenedFromExternalAppTest.java:502:
@FlakyTest(message = "http://crbug.com/6467101")
This is a buganizer bug.

[xunjieli, 2016-04-08 14:06:01]

cronet/* lgtm. You mentioned in your email that flaky tests can be tracked in
some dashboard. Is that ready? Can we have a crbug for this and the related
efforts?

[Yoland Yan(Google), 2016-04-08 20:35:51]

On 2016/04/08 14:06:01, xunjieli wrote:
> cronet/* lgtm. You mentioned in your email that flaky tests can be tracked in
> some dashboard. Is that ready? Can we have a crbug for this and the related
> efforts?

Hi Xuejieli, thanks! I created this crbug to keep track of it. crbug.com/601909.
Please let me know if you have any questions or suggestions

[no sievers, 2016-04-11 17:26:44]

content lgtm

[Yoland Yan(Google), 2016-04-11 23:24:36]

The CQ bit was checked by yolandyan@google.com

[commit-bot: I haz the power, 2016-04-11 23:25:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1856273002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1856273002/40001

[commit-bot: I haz the power, 2016-04-11 23:43:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-11 23:43:52]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Yoland Yan(Google), 2016-04-13 02:16:05]

Description was changed from

==========
Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

BUG=
==========

to

==========
Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

BUG=
==========

[Yoland Yan(Google), 2016-04-13 02:16:06]

yolandyan@google.com changed reviewers:
+ danakj@chromium.org

[danakj, 2016-04-13 23:55:57]

LGTM do you need gyp too?

[Yoland Yan(Google), 2016-04-18 16:28:24]

The CQ bit was checked by yolandyan@google.com

[Yoland Yan(Google), 2016-04-18 16:28:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org, xunjieli@chromium.org, mikecase@chromium.org,
sievers@chromium.org, dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/1856273002/#ps120001
(title: " ")

[commit-bot: I haz the power, 2016-04-18 16:28:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1856273002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1856273002/120001

[commit-bot: I haz the power, 2016-04-18 17:25:24]

Description was changed from

==========
Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

BUG=
==========

to

==========
Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

BUG=
==========

[commit-bot: I haz the power, 2016-04-18 17:25:26]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-04-18 17:26:44]

Description was changed from

==========
Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

BUG=
==========

to

==========
Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest

BUG=

Committed: https://crrev.com/c4f7db8464349b98b0c75f098b73b9eeaa863bf0
Cr-Commit-Position: refs/heads/master@{#387937}
==========

[commit-bot: I haz the power, 2016-04-18 17:26:45]

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