New issue
Advanced search Search tips

Issue 772271 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

webkit_unit_tests failing on WebKit Android (Nexus4)

Project Member Reported by nhiroki@chromium.org, Oct 6 2017

Issue description

webkit_unit_tests failing on WebKit Android (Nexus4):
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29

Looks like V8 fails to mmap for initialization:
[FATAL:v8_initializer.cc(248)] Couldn't mmap v8 natives data file, status code is 1
 
Labels: Infra-Troopers OS-Android
I cannot find a culprit CL. This could be an infra issue...?
Labels: Pri-1 Type-Bug
Status: Untriaged (was: Available)
I think it is quite likely an infra issue. Tests failing to map a file from disk and there are no relevant changes in the regression range.
First failure is at https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/70086 and this is immediately following another suspicious (but not the same) failure where device provisioning failed: https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/70085

Comment 4 by no...@chromium.org, Oct 6 2017

Components: Infra
Status: Available (was: Untriaged)
ack

Comment 5 by no...@chromium.org, Oct 6 2017

Owner: bpastene@chromium.org
Status: Assigned (was: Available)
Cc: dpranke@chromium.org brat...@opera.com
I think the first failing build is https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/70085

The provision_devices failure is a red herring. Network flake interrupted the slave's connection to the master. Had the build gotten to the point of running tests, webkit_unit_tests might have failed.

In fact, in that build's blamelist is the cl https://chromium-review.googlesource.com/c/chromium/src/+/677449 which looks pretty relevant. Could that be responsible?
Labels: -Infra-Troopers
Owner: dpranke@chromium.org
Status: Started (was: Assigned)
Seems quite plausible. I'll revert the CL and we'll see.
Labels: -Sheriff-Chromium

Comment 9 by brat...@opera.com, Oct 9 2017

Seems to be more than one issue with webkit_unit_tests. Jumbo (though disabled) seems to have added a v8 startup crash (bad compiler flags?) which the revert fixed but there is now other failures and it is hard to say exactly when they were introduced since the test crashed for several hours. Unless there is another bot which didn't crash?

70083 Scrolling test failure
70084 Successful!
70085 Exception provison devices (seems to have lost contact with the device)
70086 webkit_unit_test: Failed to list tests on device, startup crash v8
...
70132 webkit_unit_test: Failed to list tests on device, startup crash v8
jumbo revert landed 15:35
70133 webkit_unit_tests ImageDocumentTest.ImageCenteredAtDeviceScaleFactor
70134 webkit_unit_tests ImageDocumentTest.ImageCenteredAtDeviceScaleFactor
....




Yes, the now fixed mmap issue masked the ImageDocumentTest.ImageCenteredAtDeviceScaleFactor so it's hard to find the culprit of that in the build list. I'll file a separate bug for it.
Filed  issue 772860  (and found likely culprit CL).

Comment 12 by brat...@opera.com, Oct 9 2017

Status: Fixed (was: Started)
And I've found the cause of the android problem. Bad packaging in android because I incorrectly split a test() into test()+source_set().

So we're done here. Thanks for reverting it dpranke!

Sign in to add a comment