806168 - Add test for bug 802366 and bug 806127

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, defect)

Description

[Justin Lebar (not reading bugmail)]

I think this can be done by modifying the test added in bug 786295, although getting this to work is proving tricky because I don't understand anything about IndexedDB.

Comment 1

[Justin Lebar (not reading bugmail)]

Attachment: Tests, v1

Comment 2

[Justin Lebar (not reading bugmail)]

Attachment: Epilogue: Remove extra indentation in webapp_clearBrowserData.js.

The extra indentaiton was necessary because we wanted hg/git to notice that webapp_clearBrowserData.js was actually a rename of test_webapp_clearBrowserData.html.

The only non-whitespace change here is removing the comment at the top of the file saying that we're going to make this change.

Comment 3

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Comment on attachment 675968 [details] [diff] [review]
Tests, v1

Review of attachment 675968 [details] [diff] [review]:
-----------------------------------------------------------------

This looks great! Thanks!

Comment 4

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Comment on attachment 675969 [details] [diff] [review]
Epilogue: Remove extra indentation in webapp_clearBrowserData.js.

Review of attachment 675969 [details] [diff] [review]:
-----------------------------------------------------------------

Can you please add the creative commons license to this file too? Thanks!

Comment 5

[Justin Lebar (not reading bugmail)]

Landed on inbound; see bug 802366 comment 46.

Comment 6

[Justin Lebar (not reading bugmail)]

Follow-up to disable this test on Android.

https://hg.mozilla.org/integration/mozilla-inbound/rev/8bc60d1ab725

The test I copied here was already disabled on Android, so this is nothing interesting.

Comment 7

[Justin Lebar (not reading bugmail)]

And backed out for further orange; see bug 802366 comment 48.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Sorry, but this got caught up in the bug 802366 bustage and was backed out.
https://hg.mozilla.org/integration/mozilla-inbound/rev/5dc1c0530b00

Comment 9

[Justin Lebar (not reading bugmail)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7bd8f1301fce
https://hg.mozilla.org/integration/mozilla-inbound/rev/8ed3ac815205

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7bd8f1301fce
https://hg.mozilla.org/mozilla-central/rev/8ed3ac815205

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/53e7a455e3e7
https://hg.mozilla.org/releases/mozilla-aurora/rev/ca854140dd54