New issue
Advanced search Search tips

Issue 593918 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

3 ChromePublic Tests fail

Reported by huape...@amazon.com, Mar 10 2016

Issue description

Chrome Version       : Tip of master
Platform:  Android, I ran it on Nexus 7.
What steps will reproduce the problem?
(1) build chrome public test apk
(2) run ChromePublicTest
./build/android/test_runner.py instrumentation --test-apk ChromePublicTest --additional-apk ./out/Debug/apks/ChromiumNetTestSupport.apk --disable-dalvik-asserts  --num_retries 0 --debug --isolate-file-path chrome/chrome_public_test_apk.isolate -f BindingManagerIntegrationTest#testRestoreSharedRenderer:UndoTabModelTest#testOutOfOrder1:InfoBarTest#testInfoBarForPopUp

What is the expected result?
 [  PASSED  ]

What happens instead?
C   18.315s Main  [  FAILED  ] org.chromium.chrome.browser.BindingManagerIntegrationTest#testRestoreSharedRenderer (CRASHED)
C   18.302s Main  [  FAILED  ] org.chromium.chrome.browser.tabmodel.UndoTabModelTest#testOutOfOrder1 (CRASHED)
...

Please provide any additional information below. Attach a screenshot if
possible.

It is likely this change causing the problem
https://codereview.chromium.org/1732323002
Tests would pass if I revert them.
 
Can you check adb logcat output to see where it fails?

Comment 2 by huape...@amazon.com, Mar 11 2016

here is the stack trace. please see attachment.
crash_stack_trace.txt
54.3 KB View Download
Labels: OS-Android
Just in case it matters, what is the version of the Android on Nexus 7?

Comment 5 by huape...@amazon.com, Mar 11 2016

It was running on lollipop.
Cc: jbudorick@chromium.org
Status: Available (was: Unconfirmed)
Components: Tests
Labels: -Pri-3 Pri-2
we're seeing this consistently on the tablet bots on chromium.android :(

I'll find someone to look at this this week.
Cc: jinsuk...@chromium.org
(unless jinsukkim is addressing it)
Cc: -jinsuk...@chromium.org
Owner: jinsuk...@chromium.org
Status: Started (was: Available)
Working on it now
There are a couple of other issues filed for the same bug. CL is up for review crrev.com/1797663002
Project Member

Comment 12 by bugdroid1@chromium.org, Mar 22 2016

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

commit 83c640830c2f4df4b03ae18609a9dc7bd16b15c3
Author: jinsukkim <jinsukkim@chromium.org>
Date: Tue Mar 22 03:20:31 2016

Fixes failing instrumentation tests on Nakasi

BUG=593958,  593918 , 594407, 592408

Review URL: https://codereview.chromium.org/1797663002

Cr-Commit-Position: refs/heads/master@{#382495}

[modify] https://crrev.com/83c640830c2f4df4b03ae18609a9dc7bd16b15c3/content/browser/renderer_host/render_widget_host_view_android.cc

Project Member

Comment 13 by bugdroid1@chromium.org, Mar 24 2016

Labels: merge-merged-2661
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8af55e5c0b014ef0635de57c5100fdbcf4feb048

commit 8af55e5c0b014ef0635de57c5100fdbcf4feb048
Author: jinsukkim <jinsukkim@chromium.org>
Date: Thu Mar 24 01:42:51 2016

Fixes failing instrumentation tests on Nakasi

BUG=593958,  593918 , 594407, 592408

Review URL: https://codereview.chromium.org/1797663002

Cr-Commit-Position: refs/heads/master@{#382495}
(cherry picked from commit 83c640830c2f4df4b03ae18609a9dc7bd16b15c3)

NOTRY=true
NOPRESUBMIT=true

Review URL: https://codereview.chromium.org/1829653002

Cr-Commit-Position: refs/branch-heads/2661@{#372}
Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081}

[modify] https://crrev.com/8af55e5c0b014ef0635de57c5100fdbcf4feb048/content/browser/renderer_host/render_widget_host_view_android.cc

Status: Fixed (was: Started)

Sign in to add a comment