Disable BrowserTest.GetSizeForNewRenderView on Mac

Disable BrowserTest.GetSizeForNewRenderView on Mac

Broken in https://codereview.chromium.org/2404783002.

TBR=sky@chromium.org
BUG=655112
Committed: https://crrev.com/ede24242bd8b521e122812c3c703020840eac153
Cr-Commit-Position: refs/heads/master@{#424724}

[msramek, 2016-10-12 12:36:06]

The CQ bit was checked by msramek@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-12 12:36:18]

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

[msramek, 2016-10-12 13:01:20]

Description was changed from

==========
Disable BrowserTest.GetSizeForNewRenderView on Mac

Broken in https://codereview.chromium.org/2404783002.

BUG=655112
==========

to

==========
Disable BrowserTest.GetSizeForNewRenderView on Mac

Broken in https://codereview.chromium.org/2404783002.

TBR=sky@chromium.org
BUG=655112
==========

[msramek, 2016-10-12 13:01:21]

msramek@chromium.org changed reviewers:
+ erikchen@chromium.org, eugenebut@chromium.org, shrike@chromium.org,
sky@chromium.org, tapted@chromium.org, yunchao.he@intel.com

[msramek, 2016-10-12 13:03:09]

Sheriff here disabling the test broken by
https://codereview.chromium.org/2404783002

Adding author and reviewers as TBR, but since none are owners, of c/b/ui, also
adding sky@ to TBR to please review as owner.

Cheers,
Martin

[msramek, 2016-10-12 13:03:20]

The CQ bit was unchecked by msramek@chromium.org

[msramek, 2016-10-12 13:03:25]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2016-10-12 13:03:39]

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-10-12 13:17:48]

Description was changed from

==========
Disable BrowserTest.GetSizeForNewRenderView on Mac

Broken in https://codereview.chromium.org/2404783002.

TBR=sky@chromium.org
BUG=655112
==========

to

==========
Disable BrowserTest.GetSizeForNewRenderView on Mac

Broken in https://codereview.chromium.org/2404783002.

TBR=sky@chromium.org
BUG=655112
==========

[commit-bot: I haz the power, 2016-10-12 13:17:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-12 13:20:55]

Description was changed from

==========
Disable BrowserTest.GetSizeForNewRenderView on Mac

Broken in https://codereview.chromium.org/2404783002.

TBR=sky@chromium.org
BUG=655112
==========

to

==========
Disable BrowserTest.GetSizeForNewRenderView on Mac

Broken in https://codereview.chromium.org/2404783002.

TBR=sky@chromium.org
BUG=655112

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

[commit-bot: I haz the power, 2016-10-12 13:20:56]

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

[sky, 2016-10-12 16:15:11]

If you know the patch that broke the test why didn't you revert the patch?

[Eugene But (OOO till 7-30), 2016-10-12 21:40:00]

On 2016/10/12 16:15:11, sky wrote:
> If you know the patch that broke the test why didn't you revert the patch?
I guess the problem is that the patch is not revertabe as subsequent changes in
the same file has landed.