Closed
Bug 604285
Opened 15 years ago
Closed 15 years ago
category name for mobile clh should start with 'z'
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dougt, Assigned: dougt)
Details
Attachments
(1 file)
|
1.31 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
https://wiki.mozilla.org/XUL:Command_Line_Handling
The CLH are called in ABC order. Currently fennec's is 'm'. The talos tests start with 't'.
Since we want anyone to override us, we should probably be 'z', or maybe 'y' (so that addons might go after us if they really really wanted to).
|
Assignee | |
Comment 1•15 years ago
|
||
Assignee: nobody → doug.turner
Attachment #483080 -
Flags: review?(mark.finkle)
|
||
Comment 2•15 years ago
|
||
Comment on attachment 483080 [details] [diff] [review]
patch v.1
We need to be before the DefaultCLH in toolkit, which is 'y', so make us 'w'
r+ with that change
Attachment #483080 -
Flags: review?(mark.finkle) → review+
|
|
||
Comment 3•15 years ago
|
||
I meant 'x'
|
|
Assignee | |
Comment 4•15 years ago
|
||
got it!
22:14 <@mfinkle> use "x"
http://hg.mozilla.org/mobile-browser/rev/2a0e63ecb5b2
|
|
Assignee | |
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
||
Comment 5•14 years ago
|
||
How can I verify this bug?
You need to log in
before you can comment on or make changes to this bug.
Description
•