1023413 - Introduce bool nsICacheStorage.exists(nsIURI, ACString)

Status: RESOLVED FIXED

(Core :: Networking: Cache, enhancement)

Description

[Honza Bambas (:mayhemer)]

This should give a sync result.  Thinking of a tri-state logic here.  When index is not up-to-date, we could just throw, NS_ERROR_IN_PROGRESS?

Comment 1

[Honza Bambas (:mayhemer)]

Attachment: v1

Comment 2

[Carsten Book [:Tomcat]]

sorry had to back this out in https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=8c81fc70045c since it seems that one of this 3 csets cause failing xpcshell tests on android 2.2 Armv6 like this https://tbpl.mozilla.org/php/getParsedLog.php?id=42255066&tree=Mozilla-Inbound

Comment 3

[Honza Bambas (:mayhemer)]

(In reply to Carsten Book [:Tomcat] from comment #2)
> sorry had to back this out in
> https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=8c81fc70045c since it
> seems that one of this 3 csets cause failing xpcshell tests on android 2.2
> Armv6 like this
> https://tbpl.mozilla.org/php/getParsedLog.php?id=42255066&tree=Mozilla-
> Inbound

Thanks, seems like it's caused by this bug since it's the new test introduced here that is failing.  What androidity is that again..?  I will check and probably just disable on android, I assume there is some weird setting on armv6 as usual.

Comment 4

[Honza Bambas (:mayhemer)]

Forgot that mem cache is disabled on And.

https://tbpl.mozilla.org/?tree=Try&rev=df64ac05c87e

Comment 5

[Honza Bambas (:mayhemer)]

Attachment: v1.1

as v1 but disabled the new test on Android.

Comment 6

[Honza Bambas (:mayhemer)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/325e187a682a

Comment 7

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

https://hg.mozilla.org/mozilla-central/rev/325e187a682a

Comment 8

[Joshua Cranmer [:jcranmer]]

Shouldn't the test_cache2-24-exists.js be guarded with newCacheBackEndUsed() like http://dxr.mozilla.org/comm-central/source/mozilla/netwerk/test/unit/test_cache2-23-read-over-chunk.js#7 ?

Comment 9

[Honza Bambas (:mayhemer)]

(In reply to Joshua Cranmer [:jcranmer] from comment #8)
> Shouldn't the test_cache2-24-exists.js be guarded with newCacheBackEndUsed()
> like
> http://dxr.mozilla.org/comm-central/source/mozilla/netwerk/test/unit/
> test_cache2-23-read-over-chunk.js#7 ?

Bug 1032594.