New issue
Advanced search Search tips

Issue 610567 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Layout Test usb/usbDevice-iframe.html is failing

Project Member Reported by hayato@chromium.org, May 10 2016

Issue description

Comment 1 by hayato@chromium.org, May 10 2016

Owner: reillyg@chromium.org
Status: Assigned (was: Available)
Marking this as timeout in https://codereview.chromium.org/1963043002.

reillyg@, could you take a look?
Project Member

Comment 2 by bugdroid1@chromium.org, May 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b7a5c0303735cd59bb6cee5571ba646e5d1b05f8

commit b7a5c0303735cd59bb6cee5571ba646e5d1b05f8
Author: hayato <hayato@chromium.org>
Date: Tue May 10 05:21:40 2016

Mark usb/usbDevice-iframe.html as timeout

This test has been failing on all bots.

TBR=iclelland@chromium.org,stanisc@chromium.org
BUG= 610567 

Review-Url: https://codereview.chromium.org/1963043002
Cr-Commit-Position: refs/heads/master@{#392549}

[modify] https://crrev.com/b7a5c0303735cd59bb6cee5571ba646e5d1b05f8/third_party/WebKit/LayoutTests/TestExpectations

Project Member

Comment 3 by bugdroid1@chromium.org, May 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9204cc631cbf8ab6af575b65c99d70e4816b4196

commit 9204cc631cbf8ab6af575b65c99d70e4816b4196
Author: iclelland <iclelland@chromium.org>
Date: Tue May 10 15:14:49 2016

Mark usb/usbDevice-iframe.html as Flaky

This was previously marked as just timeout, but has been erroring out on
WebKit Linux (dbg) as well.

TBR=reillyg@chromium.org,hayato@chromium.org,stanisc@chromium.org
NOTRY=true
BUG= 610567 

Review-Url: https://codereview.chromium.org/1963103002
Cr-Commit-Position: refs/heads/master@{#392606}

[modify] https://crrev.com/9204cc631cbf8ab6af575b65c99d70e4816b4196/third_party/WebKit/LayoutTests/TestExpectations

Project Member

Comment 4 by bugdroid1@chromium.org, May 11 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/74030c0b688d538776300991ad778ae0b095a15e

commit 74030c0b688d538776300991ad778ae0b095a15e
Author: reillyg <reillyg@chromium.org>
Date: Wed May 11 01:04:35 2016

Bind former arrow function in connection.js to this.

https://codereview.chromium.org/1960413002 replaced an arrow function
with a regular function in connection.js however this function needs to
be explicitly bound to |this|.

BUG= 610567 

Review-Url: https://codereview.chromium.org/1971603002
Cr-Commit-Position: refs/heads/master@{#392800}

[modify] https://crrev.com/74030c0b688d538776300991ad778ae0b095a15e/mojo/public/js/connection.js
[modify] https://crrev.com/74030c0b688d538776300991ad778ae0b095a15e/third_party/WebKit/LayoutTests/TestExpectations

Status: Fixed (was: Assigned)

Comment 6 by sshru...@google.com, May 18 2016

Labels: Test-Layout

Sign in to add a comment