Disable [TranslateTestCase testTranslateInfobar]

Disable [TranslateTestCase testTranslateInfobar]

TBR=sdefresne@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2914253002
Cr-Commit-Position: refs/heads/master@{#476363}
Committed: https://chromium.googlesource.com/chromium/src/+/5fdd27d882d65b574c74fdc6997536891405aceb

[michaeldo, 2017-06-01 17:58:10]

The CQ bit was checked by michaeldo@chromium.org

[commit-bot: I haz the power, 2017-06-01 17:58:49]

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, 2017-06-01 18:34:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496339890788980, "parent_rev":
"298b9328adbfe81b92c2dadce496dac1e8ddfeb5", "commit_rev":
"5fdd27d882d65b574c74fdc6997536891405aceb"}

[commit-bot: I haz the power, 2017-06-01 18:34:10]

Description was changed from

==========
Disable [TranslateTestCase testTranslateInfobar]

TBR=sdefresne@chromium.org
BUG=none
==========

to

==========
Disable [TranslateTestCase testTranslateInfobar]

TBR=sdefresne@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2914253002
Cr-Commit-Position: refs/heads/master@{#476363}
Committed:
https://chromium.googlesource.com/chromium/src/+/5fdd27d882d65b574c74fdc69975...
==========

[commit-bot: I haz the power, 2017-06-01 18:34:12]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5fdd27d882d65b574c74fdc69975...

[sdefresne, 2017-06-02 08:15:44]

On 2017/06/01 18:34:12, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/chromium/src/+/5fdd27d882d65b574c74fdc69975...

I landed a CL yesterday to fix the test
https://chromium-review.googlesource.com/c/521108/ and sent it to the sheriff
for review.
It landed two hours before this CLs. I'm unsure why the test was disabled.

[sdefresne, 2017-06-02 08:16:06]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2918073002/ by sdefresne@chromium.org.

The reason for reverting is: The test had already been fixed when this CL
landed..

[michaeldo, 2017-06-02 14:54:17]

On 2017/06/02 08:15:44, sdefresne wrote:
> On 2017/06/01 18:34:12, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1) as
> >
>
https://chromium.googlesource.com/chromium/src/+/5fdd27d882d65b574c74fdc69975...
> 
> I landed a CL yesterday to fix the test
> https://chromium-review.googlesource.com/c/521108/ and sent it to the sheriff
> for review.
> It landed two hours before this CLs. I'm unsure why the test was disabled.

Sorry, I was the U.S. sheriff and wasn't aware of that CL to fix the test.
(Thank you for the quick fix, please be sure to CC the "other" sheriff if the
shift times are likely to overlap.) I disabled it after multiple tree failures
per your notice in IRC after seeing it was never disabled and the tree was still
failing. Your fix just missed landing by the time I disabled the test.

[sdefresne, 2017-06-02 15:28:01]

On 2017/06/02 14:54:17, michaeldo wrote:
> On 2017/06/02 08:15:44, sdefresne wrote:
> > On 2017/06/01 18:34:12, commit-bot: I haz the power wrote:
> > > Committed patchset #1 (id:1) as
> > >
> >
>
https://chromium.googlesource.com/chromium/src/+/5fdd27d882d65b574c74fdc69975...
> > 
> > I landed a CL yesterday to fix the test
> > https://chromium-review.googlesource.com/c/521108/ and sent it to the
sheriff
> > for review.
> > It landed two hours before this CLs. I'm unsure why the test was disabled.
> 
> Sorry, I was the U.S. sheriff and wasn't aware of that CL to fix the test.
> (Thank you for the quick fix, please be sure to CC the "other" sheriff if the
> shift times are likely to overlap.) I disabled it after multiple tree failures
> per your notice in IRC after seeing it was never disabled and the tree was
still
> failing. Your fix just missed landing by the time I disabled the test.

No problem. I'll put both sheriff as reviewer next time.