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

Issue 713820 link

Starred by 1 user

Issue metadata

Status: Closed
Owner:
Closed: Sep 4
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

crrev.com/25911df causes NavigatingExtensionPopupBrowserTest.DownloadViaPost to fail.

Project Member Reported by sgu...@chromium.org, Apr 20 2017

Issue description

See https://uberchromegw.corp.google.com/i/chromium.android/builders/Marshmallow%2064%20bit%20Tester

junit.framework.AssertionFailedError: Page did not load.  Tab information at time of failure -- url: null, final URL: about:blank, load progress: 100, is loading: false
	at org.chromium.chrome.test.util.ChromeTabUtils.waitForTabPageLoaded(ChromeTabUtils.java:84)
	at org.chromium.chrome.test.ChromeActivityTestCommon.startMainActivityFromIntent(ChromeActivityTestCommon.java:388)
	at org.chromium.chrome.test.ChromeActivityTestCommon.startMainActivityWithURL(ChromeActivityTestCommon.java:346)
	at org.chromium.chrome.test.ChromeActivityTestCommon.startMainActivityOnBlankPage(ChromeActivityTestCommon.java:354)
	at org.chromium.chrome.test.ChromeActivityTestCaseBase.startMainActivityOnBlankPage(ChromeActivityTestCaseBase.java:269)
	at org.chromium.chrome.browser.snackbar.DataReductionPromoSnackbarControllerTest.startMainActivity(DataReductionPromoSnackbarControllerTest.java:38)
	at org.chromium.chrome.test.ChromeActivityTestCaseBase.setUp(ChromeActivityTestCaseBase.java:56)
	at org.chromium.base.test.BaseTestResult.runParameterized(BaseTestResult.java:161)
	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:124)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:555)
	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1879)
 

Comment 1 by sgu...@chromium.org, Apr 20 2017

https://codereview.chromium.org/2740783003 is a candidate breaker. Will revert speculatively.

Comment 2 by sgu...@chromium.org, Apr 22 2017

Cc: ah...@yandex-team.ru tanderson@chromium.org
Looking at the logs now. https://uberchromegw.corp.google.com/i/chromium.android/builders/Marshmallow%2064%20bit%20Tester/builds/11408 was failing, the next one cleared that test, while still failing in another one. I don't think this is related. However, tanderson@ also tried to revert the same CL

Comment 3 by sgu...@chromium.org, Apr 22 2017

Cc: -bengr@chromium.org
Labels: -Pri-1 Pri-3
Owner: ah...@yandex-team.ru
Summary: crrev.com/25911df causes NavigatingExtensionPopupBrowserTest.DownloadViaPost to fail. (was: testDataReductionPromoSnackbarControllerCommandLineFlag is failing.)
Looking at the builder page I see that https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.11%20Tests/builds/11346 cleared the test fail which started at https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.11%20Tests/builds/11339

that corresponds with the CL landing and with my revert, and tanderson@ is likely correct.

changing the title and passing to ahest@yandex-team.ru to make sure he is aware of the invetiagion.
Can you please post the failure logs here? I dont have access to them, and can't repro.
Cc: sgu...@chromium.org
sgurun@ can you please post the logs?

Comment 6 by sgu...@chromium.org, Apr 27 2017

Hello,

[ RUN      ] NavigatingExtensionPopupBrowserTest.DownloadViaPost
2017-04-20 08:54:30.843 browser_tests[98493:1136080] NSWindow warning: adding an unknown subview: <FullSizeContentView: 0x7f860d1f1cf0>. Break on NSLog to debug.
2017-04-20 08:54:30.844 browser_tests[98493:1136080] Call stack:
(
    "+callStackSymbols disabled for performance reasons"
)
BrowserTestBase received signal: Terminated: 15. Backtrace:
0   browser_tests                       0x000000010ae9672c base::debug::StackTrace::StackTrace(unsigned long) + 28
1   browser_tests                       0x000000010b5de878 content::(anonymous namespace)::DumpStackTraceSignalHandler(int) + 200
2   libsystem_platform.dylib            0x00007fff8a9d152a _sigtramp + 26
3   ???                                 0x00007f860af763e0 0x0 + 140213686330336
4   CoreFoundation                      0x00007fff9ba92124 __CFRunLoopServiceMachPort + 212
5   CoreFoundation                      0x00007fff9ba915ec __CFRunLoopRun + 1356
6   CoreFoundation                      0x00007fff9ba90e38 CFRunLoopRunSpecific + 296
7   HIToolbox                           0x00007fff8ecf5935 RunCurrentEventLoopInMode + 235
8   HIToolbox                           0x00007fff8ecf576f ReceiveNextEventCommon + 432
9   HIToolbox                           0x00007fff8ecf55af _BlockUntilNextEventMatchingListInModeWithFilter + 71
10  AppKit                              0x00007fff889bedf6 _DPSNextEvent + 1067
11  AppKit                              0x00007fff889be226 -[NSApplication _nextEventMatchingEventMask:untilDate:inMode:dequeue:] + 454
12  browser_tests                       0x000000010afc7d40 __71-[BrowserCrApplication nextEventMatchingMask:untilDate:inMode:dequeue:]_block_invoke + 64
13  browser_tests                       0x000000010aeb53aa base::mac::CallWithEHFrame(void () block_pointer) + 10
14  browser_tests                       0x000000010afc7c84 -[BrowserCrApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 164
15  AppKit                              0x00007fff889b2d80 -[NSApplication run] + 682
16  browser_tests                       0x000000010aec54fe base::MessagePumpNSApplication::DoRun(base::MessagePump::Delegate*) + 334
17  browser_tests                       0x000000010aec4aec base::MessagePumpCFRunLoopBase::Run(base::MessagePump::Delegate*) + 92
18  browser_tests                       0x000000010aec0dfe base::MessageLoop::RunHandler() + 94
19  browser_tests                       0x000000010aee5503 base::RunLoop::Run() + 115
20  browser_tests                       0x000000010b621571 content::RunMessageLoop() + 65
21  browser_tests                       0x000000010b5e6cba content::DownloadTestObserver::WaitForFinished() + 42
22  browser_tests                       0x0000000108c203e4 extensions::(anonymous namespace)::NavigatingExtensionPopupBrowserTest_DownloadViaPost_Test::RunTestOnMainThread() + 788
23  browser_tests                       0x000000010af60cb7 InProcessBrowserTest::RunTestOnMainThreadLoop() + 215
24  browser_tests                       0x000000010b5de5d6 content::BrowserTestBase::ProxyRunTestOnMainThreadLoop() + 310
25  browser_tests                       0x000000010afcce86 ChromeBrowserMainParts::PreMainMessageLoopRunImpl() + 4358
26  browser_tests                       0x000000010afcbc7e ChromeBrowserMainParts::PreMainMessageLoopRun() + 62
27  browser_tests                       0x0000000109d72ed3 content::BrowserMainLoop::PreMainMessageLoopRun() + 67
28  browser_tests                       0x000000010a0b1bc7 content::StartupTaskRunner::RunAllTasksNow() + 39
29  browser_tests                       0x0000000109d714e2 content::BrowserMainLoop::CreateStartupTasks() + 658
30  browser_tests                       0x0000000109d756f4 content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const&) + 756
31  browser_tests                       0x0000000109d6eea4 content::BrowserMain(content::MainFunctionParams const&) + 100
32  browser_tests                       0x000000010ae7949b content::ContentMainRunnerImpl::Run() + 891
33  browser_tests                       0x000000010cb3b8e8 service_manager::Main(service_manager::MainParams const&) + 1032
34  browser_tests                       0x000000010ae787c4 content::ContentMain(content::ContentMainParams const&) + 68
35  browser_tests                       0x000000010b5de2b2 content::BrowserTestBase::SetUp() + 1906
36  browser_tests                       0x000000010af5ffc6 InProcessBrowserTest::SetUp() + 406
37  browser_tests                       0x000000010bd2d0d1 testing::Test::Run() + 97
38  browser_tests                       0x000000010bd2dc00 testing::TestInfo::Run() + 288
39  browser_tests                       0x000000010bd2e167 testing::TestCase::Run() + 263
40  browser_tests                       0x000000010bd34167 testing::internal::UnitTestImpl::RunAllTests() + 871
41  browser_tests                       0x000000010bd33dd3 testing::UnitTest::Run() + 163
42  browser_tests                       0x000000010af759b3 base::TestSuite::Run() + 163
43  browser_tests                       0x000000010ae8804f ChromeTestSuiteRunner::RunTestSuite(int, char**) + 31
44  browser_tests                       0x000000010b61e519 content::LaunchTests(content::TestLauncherDelegate*, int, int, char**) + 313
45  browser_tests                       0x000000010ae87fda main + 90
46  libdyld.dylib                       0x00007fff952f55ad start + 1
[122/601] NavigatingExtensionPopupBrowserTest.DownloadViaPost (TIMED OUT)
Thanks! I'll look into why this happens.

One question: is this bot running with PlzNavigate? Finally I've seen the failure locally but only with --enable-browser-side-navigation switch. 

Comment 8 by sgu...@chromium.org, Apr 28 2017

Cc: nasko@chromium.org
+nasko to answer comment #7

Comment 9 by nasko@chromium.org, Apr 28 2017

The bots run both with and without PlzNavigate. Looking at the logs, the test failed in browser_side_navigation_browser_tests, so indeed it was running with PlzNavigate (aka browser side navigation) enabled.
Cc: -sgu...@chromium.org
Status: Assigned (was: Untriaged)
Status: Closed (was: Assigned)
This bug is obsolete: the original commit was reverted, its successor has finally sticked and after that there were no FindIt investigations of this test.

Sign in to add a comment