1066826 - Remove Navigator::MozisLocallyAvailable API

Status: RESOLVED FIXED

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

Description

[Tanvi Vyas[:tanvi]]

It doesn't look like this is called anywhere in gecko code:
http://mxr.mozilla.org/mozilla-central/source/dom/base/Navigator.cpp#896

It is in an idl so it may or may not cause addon compatibility issues if we remove it:
http://mxr.mozilla.org/mozilla-central/source/dom/webidl/Navigator.webidl#259

Comment 1

[Boris Zbarsky [:bzbarsky]]

So more precisely:

1) This is callable by web content, but restricted to same-origin URIs.  And totally not
   standardized.
2) There is precisely one extension I see in the addons mxr that uses it.

We should either remove it entirely or make it chromeonly if we think that extension is important.

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

A google search for this didn't seem to turn up anything useful. So i think we can mark it as [ChromeOnly] at least.

Comment 3

[Jorge Villalobos [:jorgev] (he/him)]

Should be fine to remove.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Attachment: Remove navigator.mozIsLocallyAvailable API

Comment 5

[Boris Zbarsky [:bzbarsky]]

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

Comment 6

[Carsten Book [:Tomcat]]

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

Comment 7

[alexander]

I don't have major objections against this being removed (I trust that you did it for a reason).

But for the record, we've found this very useful when doing preloading of image resources. Images that may or may not already be in the users cache (using this method to test before we fire off a background prefetch).

I had also opened an issue in Chromium about this:
https://code.google.com/p/chromium/issues/detail?id=179934

Their response: "This would ideally be a more common standard, and would probably need an async API to be acceptable."

Perhaps we were the only user of this api? :)

Comment 8

[Florian Scholz (Open Web Docs)]

Release notes:
https://developer.mozilla.org/en-US/Firefox/Releases/35#Interfaces.2FAPIs.2FDOM
Reference updated:
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/mozIsLocallyAvailable