1162211 - Cache API should use sqlite private cache mode

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Ben Kelly [:bkelly, not reviewing]]

Cache uses OpenDatabaseWithFileURL() which gives us a shared sqlite cache by default.  We don't need this feature and it adds some overhead.  We should disable the shared cache by passing "cache=private" in the file URL.

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Attachment: Use sqlite private cache mode in Cache API. r=ehsan

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c7257315c37a

Comment 3

[Ben Kelly [:bkelly, not reviewing]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d3347a2c69b7

Comment 4

[Ben Kelly [:bkelly, not reviewing]]

Lots of orange on mulet in that last try.  I think its due to the other patch in the try build.  Here's a new try:

   https://treeherder.mozilla.org/#/jobs?repo=try&revision=0f1d28de4413

Comment 5

[Ben Kelly [:bkelly, not reviewing]]

Sigh.  That didn't trigger a mulet test for some reason.  Lets try this:

  https://treeherder.mozilla.org/#/jobs?repo=try&revision=aa13c8256b6e

Comment 6

[Ben Kelly [:bkelly, not reviewing]]

Last try demonstrates this patch did not cause the mulet M1 orange from previous try.  I think its good to go once the tree opens.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/bec0992e81b5

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/bec0992e81b5