Closed
Bug 1432407
Opened 8 years ago
Closed 8 years ago
Merge loops in PlacesController.onDrop
Categories
(Firefox :: Bookmarks & History, enhancement, P2)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
FIXED
Firefox 60
| Tracking | Status | |
|---|---|---|
| firefox60 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Whiteboard: [fxsearch])
Attachments
(1 file)
Now that we're removing sync transactions, we should look again at merging the loops in onDrop.
From when I wrote the code previously, I think it is easily possible.
| Comment hidden (mozreview-request) |
|
||
Comment 2•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8945590 [details]
Bug 1432407 - Merge loops in PlacesController.onDrop now that sync transactions have been removed.
https://reviewboard.mozilla.org/r/215720/#review221616
Attachment #8945590 -
Flags: review?(mak77) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7e996018bdd5
Merge loops in PlacesController.onDrop now that sync transactions have been removed. r=mak
|
||
Comment 4•8 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in
before you can comment on or make changes to this bug.
Description
•