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

Issue 821156 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Many tests failing with "No implementation found" for GURLUtils.nativeGetOrigin

Project Member Reported by paulmiller@chromium.org, Mar 12 2018

Issue description

first failure:

https://uberchromegw.corp.google.com/i/chromium.android/builders/Lollipop%20Phone%20Tester/builds/19391

log snippet:

Error in testMultipleRelationships(org.chromium.chrome.browser.browserservices.OriginVerifierTest):
java.lang.UnsatisfiedLinkError: No implementation found for java.lang.String org.chromium.net.GURLUtils.nativeGetOrigin(java.lang.String) (tried Java_org_chromium_net_GURLUtils_nativeGetOrigin and Java_org_chromium_net_GURLUtils_nativeGetOrigin__Ljava_lang_String_2)
	at org.chromium.net.GURLUtils.nativeGetOrigin(Native Method)
	at org.chromium.net.GURLUtils.getOrigin(GURLUtils.java:23)
	at org.chromium.chrome.browser.browserservices.Origin.<init>(Origin.java:22)
	at org.chromium.chrome.browser.browserservices.OriginVerifierTest.<clinit>(OriginVerifierTest.java:43)

suspected culprit:

https://chromium-review.googlesource.com/c/chromium/src/+/934289
 
Summary: Many tests failing with "No implementation found" for GURLUtils.nativeGetOrigin (was: Lollipop Phone tester failing OriginVerifierTest)
There are many other failures with the same stack and suspect on Marshmallow 64 bit Tester:

https://ci.chromium.org/buildbot/chromium.android/Marshmallow%2064%20bit%20Tester/20241
Cc: bauerb@chromium.org
correction - the suspect is the reland of the above change:

https://chromium-review.googlesource.com/958962
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 13 2018

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

commit 5a101268b136083e235be9e92e9a1e74c344552f
Author: Peter E Conn <peconn@chromium.org>
Date: Tue Mar 13 11:43:16 2018

🤝 Initialize native for tests that rely on Origin.

Origin unfortunately requires native to be loaded to use. Make sure
native is loaded for tests that use it.

Bug:  821156 
Change-Id: Ibbf79976d49e2b0beb839034d1852b3699bfbcab
Reviewed-on: https://chromium-review.googlesource.com/960022
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Commit-Queue: Peter Conn <peconn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542771}
[modify] https://crrev.com/5a101268b136083e235be9e92e9a1e74c344552f/chrome/android/javatests/src/org/chromium/chrome/browser/browserservices/OriginVerifierTest.java
[modify] https://crrev.com/5a101268b136083e235be9e92e9a1e74c344552f/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappNavigationTest.java

Comment 5 by peconn@chromium.org, Mar 13 2018

Status: Fixed (was: Assigned)

Comment 6 by peconn@chromium.org, Mar 13 2018

Issue 821376 has been merged into this issue.

Sign in to add a comment