The test newTabPagesShareProcess in ProcessesApiTest.Processes Fails for --site-per-process |
|||||||
Issue descriptionhttps://build.chromium.org/p/chromium.fyi/builders/Site%20Isolation%20Linux/builds/8360 This test expects that NTPs should always share the same process. This seems not to make any sense in --site-per-process. I will disable it now, until we figure the importance of this test or a replacement.
,
Mar 14 2016
Thanks for getting the bot green again. I would expect this test to work in --site-per-process, since that mode doesn't affect whether we consolidate instances of sites like the NTP into a single process. Indeed, creating multiple NTPs when running with --site-per-process shows them all in the same process. We'll have to take a closer look to see why the test isn't failing.
,
Mar 22 2016
This test is also flaky on Win7 Tests (dbg). Sample builds: https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/47103 https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/47085
,
Mar 23 2016
Yeah, it's not clear to me that the failures are actually related to --site-per-process, fwiw.
,
Mar 23 2016
Yeah, it seems from the builders in #3 that the test testOnUnresponsive() is flaky on Windows.
,
Mar 30 2016
Removing Sheriff-Chromium since this issue has an owner.
,
Jun 6 2016
,
Jul 29 2016
,
Aug 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7051dd3f77013b0e0bcc7447cdbcfe38aeb66a20 commit 7051dd3f77013b0e0bcc7447cdbcfe38aeb66a20 Author: afakhry <afakhry@chromium.org> Date: Tue Aug 02 21:31:16 2016 Fix newTabPageShareProces() in chrome.processes API browsertests The test used to fail in --site-per-process BUG= 594318 TEST=browser_tests --gtest_filter=ProcessesApiTest.Processes --site-per-process Review-Url: https://codereview.chromium.org/2205563002 Cr-Commit-Position: refs/heads/master@{#409329} [modify] https://crrev.com/7051dd3f77013b0e0bcc7447cdbcfe38aeb66a20/chrome/test/data/extensions/api_test/processes/api/test.js
,
Aug 3 2016
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by bugdroid1@chromium.org
, Mar 13 2016