WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
135890
IndexedDB should respect SchemeRegistry's database access setting
https://bugs.webkit.org/show_bug.cgi?id=135890
Summary
IndexedDB should respect SchemeRegistry's database access setting
Martin Hock
Reported
2014-08-13 11:18:49 PDT
As with WebSQL, IndexedDB should respect SchemeRegistry's database access setting in private browsing mode.
Attachments
patch
(1.54 KB, patch)
2014-08-13 11:29 PDT
,
Martin Hock
ap
: review+
Details
Formatted Diff
Diff
added test
(3.80 KB, patch)
2014-08-14 11:18 PDT
,
Martin Hock
ap
: review+
Details
Formatted Diff
Diff
cleaned up test
(4.11 KB, patch)
2014-08-14 13:52 PDT
,
Martin Hock
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Martin Hock
Comment 1
2014-08-13 11:29:13 PDT
Created
attachment 236537
[details]
patch
Martin Hock
Comment 2
2014-08-13 13:10:55 PDT
One of the mac-wk2 runs flaked on storage/indexeddb/mozilla/cursor-update-updates-indexes.html but mac-wk2 is running ML and this test is somewhat flaky there:
http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=storage%2Findexeddb%2Fmozilla%2Fcursor-update-updates-indexes.html
I'll run it a few more times locally.
Alexey Proskuryakov
Comment 3
2014-08-13 13:28:20 PDT
Comment on
attachment 236537
[details]
patch Looks fine. I don't like the function name (isContextValid), but it wasn't particularly appropriate before this change either. Please add a regression test ASAP (could be in a separate bug if this is urgent to land).
Martin Hock
Comment 4
2014-08-14 11:18:27 PDT
Created
attachment 236604
[details]
added test
Alexey Proskuryakov
Comment 5
2014-08-14 12:44:09 PDT
Comment on
attachment 236604
[details]
added test View in context:
https://bugs.webkit.org/attachment.cgi?id=236604&action=review
> LayoutTests/storage/indexeddb/private-browsing-affects-indexeddb.html:7 > +if (window.testRunner) { > + testRunner.dumpAsText(); > +}
You could have used js-test harness to get this and logging for free.
> LayoutTests/storage/indexeddb/private-browsing-affects-indexeddb.html:23 > + if (!indexedDB.open('test1')) { > + log("Could not open test IndexedDB, exiting test."); > + return; > + }
I'm not sure if this is useful. We have all the other tests to verify that IndexedDB works, and dynamic changes to private browsing mode are not really meaningful.
Martin Hock
Comment 6
2014-08-14 13:52:29 PDT
Created
attachment 236616
[details]
cleaned up test
Martin Hock
Comment 7
2014-08-14 14:05:08 PDT
Committed
r172603
: <
http://trac.webkit.org/changeset/172603
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug