New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 725455 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: 2017-08-30
OS: Linux
Pri: 1
Type: Bug


Participants' hotlists:
NFC


Sign in to add a comment

NFC webkit_tests failing on 2 Linux builders

Project Member Reported by cfroussios@chromium.org, May 23 2017

Issue description

webkit_tests failing on 2 builders

Builders failed on: 

- WebKit Linux Trusty:   https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty

- WebKit Linux Trusty Leak:   https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak

Haven't found the culprit :(



 
Attached stdout of webkit_tests failure
dump.txt
8.0 MB View Download
Labels: Pri-1
Project Member

Comment 3 by bugdroid1@chromium.org, May 23 2017

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

commit 80edb6a5a953a601805b86eddda3a561f0763a7f
Author: cfroussios <cfroussios@chromium.org>
Date: Tue May 23 12:19:53 2017

Disable failing nfc webkit tests
to unblock the tree

BUG= 725455 
TBR=device-dev@chromium.org
NOTRY=True

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

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

Project Member

Comment 4 by bugdroid1@chromium.org, May 23 2017

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

commit 537653e94532216958db3c24c573c6f833cdbcbf
Author: cfroussios <cfroussios@chromium.org>
Date: Tue May 23 13:36:30 2017

Disable nfc webkit tests

They are failing with timeout on linux.

TBRing the most recent author

BUG= 725455 
TBR=shalamov
NOTRY=TRUE

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

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

Labels: -Sheriff-Chromium OS-Linux
Owner: mattreynolds@chromium.org
Status: Assigned (was: Available)
Assigned to random member of the device team. Please help triage further.
Components: Blink>NFC
Labels: Type-Bug
Owner: ----
Status: Available (was: Assigned)
Summary: NFC webkit_tests failing on 2 Linux builders (was: webkit_tests failing on 2 builders)

Comment 7 by scheib@chromium.org, Aug 16 2017

Cc: rijubrat...@intel.com alexande...@intel.com
NextAction: 2017-08-30
Owner: alexande...@intel.com
Status: Untriaged (was: Available)
Alexander, would you help look into these items please?
Status: Started (was: Untriaged)
scheib@ sure I will take a look. Thanks.
Cc: -rijubrat...@intel.com -alexande...@intel.com reillyg@chromium.org
Cannot reproduce error locally.

reilly@ I think you fixed the issue by migrating code to new Mojo JS bindings system. https://chromium-review.googlesource.com/c/584853

I removed expected failures, linux bots seems to be green. https://chromium-review.googlesource.com/c/618723
Status: Fixed (was: Started)
Tests appear to be all green on the flakiness dashboard.
Project Member

Comment 11 by bugdroid1@chromium.org, Aug 18 2017

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

commit b980223652c00b6a9e43fa1d296119e000206a1e
Author: Alexander Shalamov <alexander.shalamov@intel.com>
Date: Fri Aug 18 07:04:12 2017

[webnfc] Re-enable Web NFC tests

Failures cannot be reproduced in linux environment after
https://chromium-review.googlesource.com/c/584853 CL.

Bug:  725455 
Change-Id: Id9086169fc01115a55cec105f7f00a0820c86593
Reviewed-on: https://chromium-review.googlesource.com/618723
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Alexander Shalamov <alexander.shalamov@intel.com>
Cr-Commit-Position: refs/heads/master@{#495492}
[modify] https://crrev.com/b980223652c00b6a9e43fa1d296119e000206a1e/third_party/WebKit/LayoutTests/TestExpectations

The NextAction date has arrived: 2017-08-30
cfroussios@ could you please verify that issue is fixed. Thanks.

Sign in to add a comment