New issue
Advanced search Search tips

Issue 623265 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

OOPIF: http/tests/inspector/indexeddb/resources-panel.html is failing with --site-per-process

Project Member Reported by alex...@chromium.org, Jun 25 2016

Issue description

Repro steps:

third_party/WebKit/Tools/Scripts/run-webkit-tests -t gn-debug --additional-driver-flag=--site-per-process http/tests/inspector/indexeddb/resources-panel.html --driver-logging --full-results-html

Looks like this test navigates the main page to a different origin (http://localhost:8000/inspector/indexeddb/resources/without-indexed-db.html), but then can't handle the new page loading in a different process.  The test's callback for the navigation, navigatedAway(), never gets called after the InspectorTest.navigate(withoutIndexedDBURL, navigatedAway) call.

This test is already disabled for --site-per-process.  It works with --isolate-sites-for-testing=*.is (we can't use --isolate-extensions to run layout tests since content/ doesn't know about it, but this flag exercises the same paths).
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 25 2016

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

commit 8e9dc9562c7791803a95f84d89dc9a046e1a24b9
Author: alexmos <alexmos@chromium.org>
Date: Sat Jun 25 06:28:14 2016

Add bug numbers for remaining site-per-process layout test failures.

BUG= 623210 , 623265 , 623268 
TBR=lukasza@chromium.org

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

[modify] https://crrev.com/8e9dc9562c7791803a95f84d89dc9a046e1a24b9/third_party/WebKit/LayoutTests/FlagExpectations/site-per-process

Project Member

Comment 2 by sheriffbot@chromium.org, Jun 26 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 4 2017

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

commit 62db39d8dec89187c2518d0046b7ca28d9fc747d
Author: Lukasz Anforowicz <lukasza@chromium.org>
Date: Wed Oct 04 23:23:03 2017

Disable exceptions for tests that have "healed" themselves.

Bug:  758075 ,  700535 ,  582522 ,  602497 ,  616905 
Bug:  623268 ,  623265 ,  678492 
Change-Id: I1bdc648c9aeb97cc1d16b0d8693d04e0c76030de
Reviewed-on: https://chromium-review.googlesource.com/700983
Commit-Queue: Ɓukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506557}
[modify] https://crrev.com/62db39d8dec89187c2518d0046b7ca28d9fc747d/third_party/WebKit/LayoutTests/FlagExpectations/site-per-process

Status: Fixed (was: Untriaged)
Both Site Isolation FYI bots are green - closing this as fixed.

Sign in to add a comment