Closed
Bug 525160
Opened 15 years ago
Closed 15 years ago
Remove cmd_search and related code
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
fennec1.0
People
(Reporter: mfinkle, Assigned: mfinkle)
Details
Attachments
(1 file, 1 obsolete file)
3.60 KB,
patch
|
vingtetun
:
review+
|
Details | Diff | Splinter Review |
We don't seem to use cmd_search and BrowserUI.search anymore. Let's remove them and any support code and DTDs
Assignee | ||
Comment 1•15 years ago
|
||
Removes all the bits
Assignee: nobody → mark.finkle
Attachment #412861 -
Flags: review?(21)
Comment 2•15 years ago
|
||
Don't you also want to remove BrowserUI.search function? http://mxr.mozilla.org/mobile-browser/source/chrome/content/browser-ui.js#489
Assignee | ||
Comment 3•15 years ago
|
||
Yes, I do
Assignee | ||
Comment 4•15 years ago
|
||
Removes the BrowserUI.search method too
Attachment #412861 -
Attachment is obsolete: true
Attachment #412867 -
Flags: review?(21)
Attachment #412861 -
Flags: review?(21)
Attachment #412867 -
Flags: review?(21) → review+
Assignee | ||
Comment 5•15 years ago
|
||
pushed: https://hg.mozilla.org/mobile-browser/rev/cf5f71e6a6a0
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Post-B5
You need to log in
before you can comment on or make changes to this bug.
Description
•