Closed Bug 790168 Opened 7 years ago Closed 7 years ago

dragstart handler cleanup

Categories

(Firefox :: Tabbed Browser, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 18

People

(Reporter: dao, Assigned: dao)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
- get the browser only once

- remove the (as far as I can tell) unneeded "about:blank" fallback for currentURI

- stop special-casing pinned tabs for tabOffsetX, since the tab strip expands behind pinned tabs (bug 786291)
Attachment #659999 - Flags: review?(ttaubert)
Comment on attachment 659999 [details] [diff] [review]
patch

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

Looks good to me! Sorry, I didn't immediately understand the last chunk of the patch.
Attachment #659999 - Flags: review?(ttaubert) → review+
https://hg.mozilla.org/mozilla-central/rev/bd455398388d
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
You need to log in before you can comment on or make changes to this bug.