Bug 155581 - Move IndexedDB regular test to web-platform-tests
Summary: Move IndexedDB regular test to web-platform-tests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-03-17 02:15 PDT by youenn fablet
Modified: 2016-03-18 03:17 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.65 MB, patch)
2016-03-17 02:46 PDT, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2016-03-17 02:15:19 PDT
Following on bug 148713 discussion, it might be good to move regular IndexedDB tests to web-platform-tests folder.
That will allow updating these tests according W3C WPT repo.
Comment 1 youenn fablet 2016-03-17 02:46:02 PDT
Created attachment 274274 [details]
Patch
Comment 2 Darin Adler 2016-03-17 09:08:15 PDT
Comment on attachment 274274 [details]
Patch

Seems fine.
Comment 3 Alexey Proskuryakov 2016-03-17 09:41:54 PDT
>  Modified also IndexedDB-private-browsing/support.js to always activate testRunner private browsing mode.

I still think that it would be cleaner to enable private browsing in .html. It was extremely surprising to me to find that hidden in support.js.

At the very least, support.js should be renamed to indicate that it's different from support.js - simply having it in a different directory is too subtle.
Comment 4 youenn fablet 2016-03-17 09:56:00 PDT
(In reply to comment #3)
> >  Modified also IndexedDB-private-browsing/support.js to always activate testRunner private browsing mode.
> 
> I still think that it would be cleaner to enable private browsing in .html.
> It was extremely surprising to me to find that hidden in support.js.
> 
> At the very least, support.js should be renamed to indicate that it's
> different from support.js - simply having it in a different directory is too
> subtle.

If we add a script dedicated to those tests, the script should probably do it that way.

In the meantime, I prefer keeping all html files exactly the same as their WPT version.

Once this patch is landed, I plan to update the WPT IndexedDB tests and those tests according W3C WPT repo. Not touching the html files will make this resyncing much easier.
Comment 5 WebKit Commit Bot 2016-03-18 03:17:01 PDT
Comment on attachment 274274 [details]
Patch

Clearing flags on attachment: 274274

Committed r198394: <http://trac.webkit.org/changeset/198394>
Comment 6 WebKit Commit Bot 2016-03-18 03:17:04 PDT
All reviewed patches have been landed.  Closing bug.