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

Issue 673680 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug

Blocked on:
issue 673433



Sign in to add a comment

"org.chromium.chrome.browser.externalnav.ExternalNavigationDelegateImplTest#testIsPackageSpecializeHandler_withEphemeralResolver" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Dec 13 2016

Issue description

"org.chromium.chrome.browser.externalnav.ExternalNavigationDelegateImplTest#testIsPackageSpecializeHandler_withEphemeralResolver" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyigELEgVGbGFrZSJ_b3JnLmNocm9taXVtLmNocm9tZS5icm93c2VyLmV4dGVybmFsbmF2LkV4dGVybmFsTmF2aWdhdGlvbkRlbGVnYXRlSW1wbFRlc3QjdGVzdElzUGFja2FnZVNwZWNpYWxpemVIYW5kbGVyX3dpdGhFcGhlbWVyYWxSZXNvbHZlcgw.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 

Comment 1 by yutak@chromium.org, Dec 14 2016

Labels: OS-Android
Owner: mariakho...@chromium.org
Status: Assigned (was: Untriaged)
mariakhomenko: Can you own this?

Meanwhile, I'll disable this test.
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 14 2016

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

commit b8f7a8ed1ab30e5148c91786c69005f5d9716b63
Author: yutak <yutak@chromium.org>
Date: Wed Dec 14 06:03:58 2016

Disable org.chromium.chrome.browser.externalnav.ExternalNavigationDelegateImplTest#testIsPackageSpecializeHandler_withEphemeralResolver.

BUG= 673680 
TBR=mariakhomenko@chromium.org

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

[modify] https://crrev.com/b8f7a8ed1ab30e5148c91786c69005f5d9716b63/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java

Components: Tests>Disabled
Labels: -Sheriff-Chromium
Blockedon: 673433
Cc: mariakho...@chromium.org
Owner: jbudorick@chromium.org
Another one of these:

Device(06c330d8006b0000) 12-12 15:19:10.349 19675 19675 W System.err: java.lang.AssertionError
Device(06c330d8006b0000) 12-12 15:19:10.349 19675 19675 W System.err:   at org.chromium.chrome.browser.metrics.UmaUtils.getForegroundStartTime(UmaUtils.java:96)
Device(06c330d8006b0000) 12-12 15:19:10.349 19675 19675 W System.err:   at org.chromium.chrome.browser.tab.TabWebContentsObserver.didCommitProvisionalLoadForFrame(TabWebContentsObserver.java:211)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at org.chromium.content.browser.webcontents.WebContentsObserverProxy.didCommitProvisionalLoadForFrame(WebContentsObserverProxy.java:
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at org.chromium.base.SystemMessageHandler.nativeDoRunLoopOnce(Native Method)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at org.chromium.base.SystemMessageHandler.handleMessage(SystemMessageHandler.java:41)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at android.os.Handler.dispatchMessage(Handler.java:102)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at android.os.Looper.loop(Looper.java:136)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at android.app.ActivityThread.main(ActivityThread.java:5001)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at java.lang.reflect.Method.invokeNative(Native Method)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at java.lang.reflect.Method.invoke(Method.java:515)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:785)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:601)
Device(06c330d8006b0000) 12-12 15:19:10.359 19675 19675 W System.err:   at dalvik.system.NativeStart.main(Native Method)
Device(06c330d8006b0000) 12-12 15:19:10.369 19675 19677 D dalvikvm: GC_CONCURRENT freed 297K, 2% free 17769K/18100K, paused 2ms+3ms, total 17ms
Device(06c330d8006b0000) 12-12 15:19:10.389 19675 19675 F chromium: [FATAL:jni_android.cc(243)] Please include Java exception stack in crash report

As elsewhere, I suspect this was caused by an infrastructure bug I introduced, not by flakiness in the test. Reenabling the test.
Project Member

Comment 5 by bugdroid1@chromium.org, Dec 14 2016

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

commit 7fde98c771cafc15ea4d5c8018084b5cd64d3200
Author: jbudorick <jbudorick@chromium.org>
Date: Wed Dec 14 14:06:34 2016

Revert of Disable org.chromium.chrome.browser.externalnav.ExternalNavigationDelegateImplTest#testIsPackageSpe… (patchset #2 id:20001 of https://codereview.chromium.org/2573443004/ )

Reason for revert:
reenabling the test

Original issue's description:
> Disable org.chromium.chrome.browser.externalnav.ExternalNavigationDelegateImplTest#testIsPackageSpecializeHandler_withEphemeralResolver.
>
> BUG= 673680 
> TBR=mariakhomenko@chromium.org
>
> Committed: https://crrev.com/b8f7a8ed1ab30e5148c91786c69005f5d9716b63
> Cr-Commit-Position: refs/heads/master@{#438445}

TBR=mariakhomenko@chromium.org,yutak@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 673680 

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

[modify] https://crrev.com/7fde98c771cafc15ea4d5c8018084b5cd64d3200/chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java

Status: WontFix (was: Assigned)
Components: Tests>Disabled
Labels: Test-Disabled

Sign in to add a comment