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

Issue 649022 link

Starred by 2 users
Status: Fixed
Owner:
sa...@chromium.org
Closed: Sep 2016
Cc:
█ roc...@chromium.org
█ yzshen@chromium.org
sa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

InterfaceRegistryTest#testGetInterface is flaky even with retry

Project Member Reported by boliu@chromium.org, Sep 21 2016 
http://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyTgsSBUZsYWtlIkNvcmcuY2hyb21pdW0uY29udGVudC5icm93c2VyLkludGVyZmFjZVJlZ2lzdHJ5VGVzdCN0ZXN0R2V0SW50ZXJmYWNlDA

C  197.725s Main  [FAIL] org.chromium.content.browser.InterfaceRegistryTest#testGetInterface:
C  197.725s Main  junit.framework.AssertionFailedError: expected:<42.0> but was:<21.0>
C  197.725s Main  	at org.chromium.content.browser.InterfaceRegistryTest.testGetInterface(InterfaceRegistryTest.java:149)
C  197.725s Main  	at java.lang.reflect.Method.invokeNative(Native Method)
C  197.725s Main  	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
C  197.725s Main  	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
C  197.726s Main  	at android.test.ActivityInstrumentationTestCase2.runTest(ActivityInstrumentationTestCase2.java:192)
C  197.726s Main  	at org.chromium.content_shell_apk.ContentShellTestBase.runTest(ContentShellTestBase.java:233)
C  197.726s Main  	at org.chromium.base.test.BaseTestResult.runParameterized(BaseTestResult.java:161)
C  197.726s Main  	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:124)
C  197.726s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
C  197.726s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
C  197.726s Main  	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
C  197.726s Main  	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)

Comment 1 by yzshen@chromium.org, Sep 21 2016

Cc: roc...@chromium.org yzshen@chromium.org sa...@chromium.org

Comment 2 by roc...@chromium.org, Sep 21 2016

Owner: sa...@chromium.org
Status: Assigned (was: Available)
If this flake is new, I would guess it's somehow caused by https://codereview.chromium.org/2214383002. Could you take a look and see if that's the case Sam?

Comment 3 by boliu@chromium.org, Sep 21 2016 

earliest flake from try-flakes is from aug 4, that's like a week after the test was renamed to this name in https://codereview.chromium.org/2191033002

so not that new

no idea if it's the same error back in august though, as logs are not archived..
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 21 2016 

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

commit b0625dd0edca05cf08217d1329282c8b12d702f0
Author: boliu <boliu@chromium.org>
Date: Wed Sep 21 17:47:37 2016

Disable flaky InterfaceRegistryTest#testGetInterface

BUG= 649022 
TBR=tedchoc@chromium.org

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

[modify] https://crrev.com/b0625dd0edca05cf08217d1329282c8b12d702f0/content/public/android/javatests/src/org/chromium/content/browser/InterfaceRegistryTest.java
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 30 2016 

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

commit 86ca7f13cfcf3a15a68133a270d04bec769aa2d1
Author: sammc <sammc@chromium.org>
Date: Fri Sep 30 02:00:23 2016

Remove the JNI-based InterfaceRegistry.

Java interface registration has been migrated to a Java-implemented
InterfaceRegistry. This CL removes the now unused JNI-based
InterfaceRegistry.

BUG= 634568 , 637174 , 649022 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/browser/BUILD.gn
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/browser/android/browser_jni_registrar.cc
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/browser/android/interface_provider_android_impl.cc
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/browser/android/interface_provider_android_impl.h
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/browser/android/interface_registry_android_impl.cc
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/browser/android/interface_registry_android_impl.h
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/browser/frame_host/render_frame_host_impl.cc
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/browser/frame_host/render_frame_host_impl.h
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/browser/mojo/interface_registrar_android.cc
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/browser/mojo/interface_registrar_android.h
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/browser/renderer_host/render_process_host_impl.h
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/public/android/BUILD.gn
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/public/android/java/src/org/chromium/content/browser/InterfaceProvider.java
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/public/android/java/src/org/chromium/content/browser/InterfaceRegistrar.java
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/public/android/java/src/org/chromium/content/browser/InterfaceRegistry.java
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/public/android/javatests/src/org/chromium/content/browser/InterfaceRegistryTest.java
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/public/browser/BUILD.gn
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/public/browser/android/interface_provider_android.h
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/public/browser/android/interface_registry_android.h
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/shell/BUILD.gn
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/shell/android/BUILD.gn
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/shell/android/java/src/org/chromium/content_shell/ShellMojoTestUtils.java
[modify] https://crrev.com/86ca7f13cfcf3a15a68133a270d04bec769aa2d1/content/shell/android/shell_jni_registrar.cc
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/shell/browser/shell_mojo_test_utils_android.cc
[delete] https://crrev.com/cebea890c1e2a49072e0e5ec2bd8823ae2bb2a86/content/shell/browser/shell_mojo_test_utils_android.h

Comment 6 by sa...@chromium.org, Sep 30 2016

Status: Fixed (was: Assigned)
This test has been removed.

Comment 7 by dpranke@chromium.org, Jan 24 2018

Components: Tests>Disabled
Labels: Test-Disabled
►

Sign in to add a comment