Skip failing LayoutTest.

Skip failing LayoutTest.

Still failing after updating the expectation to Failure Pass Timeout. Skipping the test for now.

See crbug for more details.

BUG=720511
R=thakis@chromium.org

Review-Url: https://codereview.chromium.org/2879693002 .
Cr-Commit-Position: refs/heads/master@{#471110}
Committed: https://chromium.googlesource.com/chromium/src/+/c3446d5d8214478acc171910990e3562eb0bee50

[apacible, 2017-05-11 21:20:40]

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

[commit-bot: I haz the power, 2017-05-11 21:21:47]

Dry run: CQ is trying da patch.

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

[apacible, 2017-05-11 21:22:11]

Description was changed from

==========
skip utf8

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

to

==========
Skip failing LayoutTest.

Still failing after updating the expectation to Failure Pass Timeout. Skipping
the test for now.

See crbug for more details.

BUG=720511
TBR=thakis@chromium.org
==========

[apacible, 2017-05-11 21:22:11]

apacible@chromium.org changed reviewers:
+ thakis@chromium.org

[Nico, 2017-05-11 22:53:30]

lgtm, but I wonder if the old CL just took a while to take. I just had 2 of my
CLs go in. So maybe this change isn't needed after all.

(On the links I pasted on the bug, I did check that the commit position was
after your earlier TestExpectations change though. So no idea what's going on.)

[Nico, 2017-05-11 22:59:38]

On the third hand, it failed on
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...
too and that started 2h ago. So this is probably still needed.

On Thu, May 11, 2017 at 6:53 PM, <thakis@chromium.org> wrote:

> lgtm, but I wonder if the old CL just took a while to take. I just had 2
> of my
> CLs go in. So maybe this change isn't needed after all.
>
> (On the links I pasted on the bug, I did check that the commit position was
> after your earlier TestExpectations change though. So no idea what's going
> on.)
>
> https://codereview.chromium.org/2879693002/
>

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

[Nico, 2017-05-11 22:59:39]

On the third hand, it failed on
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...
too and that started 2h ago. So this is probably still needed.

On Thu, May 11, 2017 at 6:53 PM, <thakis@chromium.org> wrote:

> lgtm, but I wonder if the old CL just took a while to take. I just had 2
> of my
> CLs go in. So maybe this change isn't needed after all.
>
> (On the links I pasted on the bug, I did check that the commit position was
> after your earlier TestExpectations change though. So no idea what's going
> on.)
>
> https://codereview.chromium.org/2879693002/
>

-- 
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.

[apacible, 2017-05-11 23:08:14]

Description was changed from

==========
Skip failing LayoutTest.

Still failing after updating the expectation to Failure Pass Timeout. Skipping
the test for now.

See crbug for more details.

BUG=720511
TBR=thakis@chromium.org
==========

to

==========
Skip failing LayoutTest.

Still failing after updating the expectation to Failure Pass Timeout. Skipping
the test for now.

See crbug for more details.

BUG=720511
R=thakis@chromium.org

Review-Url: https://codereview.chromium.org/2879693002 .
Cr-Commit-Position: refs/heads/master@{#471110}
Committed:
https://chromium.googlesource.com/chromium/src/+/c3446d5d8214478acc171910990e...
==========

[apacible, 2017-05-11 23:08:15]

Committed patchset #1 (id:1) manually as
c3446d5d8214478acc171910990e3562eb0bee50 (presubmit successful).