703098 - Remove the url.file.exist check in BrowserCLH.js

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect, P3)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Bug 701483 added better file URL resolution, but we should try to optimize the code a bit. Removing unneeded file.exist checks should speed up the code.

Comment 1

[Doug Turner (:dougt)]

Attachment: patch v.1

stats are bad.

Comment 2

[Doug Turner (:dougt)]

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

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 575252 [details] [diff] [review]
patch v.1


> function resolveURIInternal(aCmdLine, aArgument) {
>   let uri = aCmdLine.resolveURI(aArgument);
> 

Remove this blank line

>+  if (uri)
>       return uri;

2 space indent

r+ with nits fixed

Comment 4

[Doug Turner (:dougt)]

https://hg.mozilla.org/projects/birch/rev/ca12f1337445