Closed Bug 703098 Opened 8 years ago Closed 8 years ago

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

Categories

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

x86
Linux
defect

Tracking

()

RESOLVED FIXED
Tracking Status
firefox11 --- fixed
fennec 11+ ---

People

(Reporter: mfinkle, Assigned: dougt)

Details

Attachments

(1 file)

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.
Assignee: nobody → doug.turner
Priority: -- → P3
Attached patch patch v.1Splinter Review
stats are bad.
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+
https://hg.mozilla.org/projects/birch/rev/ca12f1337445
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
You need to log in before you can comment on or make changes to this bug.