Closed
Bug 371439
Opened 17 years ago
Closed 17 years ago
Some old history & bookmarks frontend code is still built for places/places-bookmarks builds
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 3 alpha3
People
(Reporter: asaf, Assigned: asaf)
References
Details
Attachments
(2 files)
|
4.87 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
|
4.58 KB,
patch
|
Details | Diff | Splinter Review |
Some old history & bookmarks frontend code is still built for places/places-bookmarks builds. We should #ifdef out this so we can remove all obsoleted code once Places is shipped.
Attachment #256221 -
Flags: review?(gavin.sharp)
|
||
Comment 1•17 years ago
|
||
Comment on attachment 256221 [details] [diff] [review] patch rs=me (I haven't tested this)
Attachment #256221 -
Flags: review?(gavin.sharp) → review+
|
Assignee | |
Comment 2•17 years ago
|
||
mozilla/browser/base/content/browser-sets.inc 1.96 mozilla/browser/base/content/browser.xul 1.339 mozilla/browser/base/content/macBrowserOverlay.xu 1.15
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
||
Updated•17 years ago
|
Flags: in-testsuite-
|
||
Comment 3•10 years ago
|
||
|
|
||
Updated•10 years ago
|
Assignee: mano → rnewman
|
|
||
Comment 4•10 years ago
|
||
OK, that's the weirdest every `hg bzexport` bug I've ever seen. This patch, as you might expect, was not in my queue.
Assignee: rnewman → mano
|
|
||
Comment 5•10 years ago
|
||
You had me very confused for a moment! Was headed here to r- :)
You need to log in
before you can comment on or make changes to this bug.
Description
•