685415 - nsFileProtocolHandler::ReadURLFile is unnecessary on Android

Status: RESOLVED WONTFIX

(Core :: Networking: File, defect)

Description

[Makoto Kato [:m_kato]]

Android should return NS_ERROR_NOT_AVAILABLE.

Comment 1

[Makoto Kato [:m_kato]]

Attachment: fix

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

(In reply to Makoto Kato from comment #0)
> Android should return NS_ERROR_NOT_AVAILABLE.

why?

Comment 3

[Makoto Kato [:m_kato]]

(In reply to Brad Lassey [:blassey] from comment #2)
> (In reply to Makoto Kato from comment #0)
> > Android should return NS_ERROR_NOT_AVAILABLE.
> 
> why?

Why should we support .desktop file on Android?  .desktop is for freedesktop.org.

Comment 4

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 566750 [details] [diff] [review]
fix

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

clearing the review until my question is answered

Comment 5

[Makoto Kato [:m_kato]]

(In reply to Brad Lassey [:blassey] from comment #4)
> Comment on attachment 566750 [details] [diff] [review] [diff] [details] [review]
> fix
> 
> Review of attachment 566750 [details] [diff] [review] [diff] [details] [review]:
> -----------------------------------------------------------------
> 
> clearing the review until my question is answered

Brad and Doug, "*.desktop" file doesn't support on Android.  This code is for .desktop file handling.  You don't answer my question why "*.desktop" file has to support on Android.

Android dekstop doesn't have url shortcut file.  So we should remove this implementation on Android.

So we should return any error on this method.  If we should support "*.desktop" file, please resolved as WONTFIX.