Closed Bug 1160581 Opened 10 years ago Closed 9 years ago

Avoid using try/catch and nsIURI in frame utils

Categories

(DevTools :: Performance Tools (Profiler/Timeline), defect)

Product:
DevTools
Component:
Performance Tools (Profiler/Timeline)
Version:
37 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: jsantell, Assigned: jsantell)

References

Details

This is a bottleneck.
Assignee: nobody → jsantell
Blocks: perf-perf
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Gonna make this a dependency actually because this involves just nsIURI
Status: RESOLVED → REOPENED
Depends on: 1160691
Resolution: DUPLICATE → ---
There is arguably no difference between the two. Node parsing may be slightly faster, but even with a giant octane benchmark, we're looking at 50ms differences.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 9 years ago
Resolution: --- → FIXED
Resolution: FIXED → INVALID
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.