The default bug view has changed. See this FAQ.

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

RESOLVED FIXED

Status

()

Firefox for Android
General
P3
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mfinkle, Assigned: dougt)

Tracking

unspecified
x86
Linux
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment)

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.
(Reporter)

Updated

5 years ago
Assignee: nobody → doug.turner
Priority: -- → P3
(Assignee)

Comment 1

5 years ago
Created attachment 575252 [details] [diff] [review]
patch v.1

stats are bad.
(Assignee)

Comment 2

5 years ago
https://tbpl.mozilla.org/?tree=Try&rev=bf73e12bcbb2
(Assignee)

Updated

5 years ago
Attachment #575252 - Flags: review?(mark.finkle)
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
Attachment #575252 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/projects/birch/rev/ca12f1337445
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.