Bug 153179 – Modern IDB: Add private-browsing variant for many IDB tests, and enable private browsing in Modern IDB

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Brady Eidson]

Modern IDB: Add private-browsing variant for IDB tests that already have split HTML/JS formatting

Since we'll have two separate backends (In-memory and SQLite) and will need to test both, we need a "normal" browsing HTML and private browsing HTML file for each test.

https://bugs.webkit.org/show_bug.cgi?id=153178 covers making all of the tests be split, but I'd like to make the private browsing copies now for each test that is already split; Such test coverage should be good enough to get the SQLite backend working.

[Brady Eidson]

Retitled:
Modern IDB: Add private-browsing variant for many IDB tests, and enable private browsing in Modern IDB

[Brady Eidson]

Created attachment 269237 [details]
Patch v1

[WebKit Commit Bot]

Attachment 269237 [details] did not pass style-queue:


WARNING: Not running on native Windows.
ERROR: LayoutTests/platform/wk2/TestExpectations:767:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:824:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:825:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:826:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:827:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:828:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:839:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:872:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/wk2/TestExpectations:881:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 9 in 308 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Darin Adler]

Comment on attachment 269237 [details]
Patch v1

View in context: https://bugs.webkit.org/attachment.cgi?id=269237&action=review

> LayoutTests/resources/js-test.js:10
> +		testRunner.setPrivateBrowsingEnabled(true);

tab here, should be 4 spaces

> Source/WebCore/ChangeLog:8
> +        Tests: storage/indexeddb/aborted-versionchange-closes-private.html

I don’t think this list of tests is valuable.

[Brady Eidson]

(In reply to comment #4)
> Comment on attachment 269237 [details]
> Patch v1
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=269237&action=review
> 
> > LayoutTests/resources/js-test.js:10
> > +		testRunner.setPrivateBrowsingEnabled(true);
> 
> tab here, should be 4 spaces

Whoops!
 
> > Source/WebCore/ChangeLog:8
> > +        Tests: storage/indexeddb/aborted-versionchange-closes-private.html
> 
> I don’t think this list of tests is valuable.

Agreed.

Thanks!

[Brady Eidson]

Created attachment 269252 [details]
Patch for EWS then landing

I'll watch the EWS run on this then CQ+ once it's good.

[WebKit Commit Bot]

Comment on attachment 269252 [details]
Patch for EWS then landing

Clearing flags on attachment: 269252

Committed r195247: <http://trac.webkit.org/changeset/195247>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.