Closed Bug 604285 Opened 9 years ago Closed 9 years ago

category name for mobile clh should start with 'z'

Categories

(Firefox for Android Graveyard :: General, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dougt, Assigned: dougt)

Details

Attachments

(1 file)

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).
Attached patch patch v.1Splinter Review
Assignee: nobody → doug.turner
Attachment #483080 - Flags: review?(mark.finkle)
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+
got it!

22:14 <@mfinkle> use "x"

http://hg.mozilla.org/mobile-browser/rev/2a0e63ecb5b2
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
How can I verify this bug?
You need to log in before you can comment on or make changes to this bug.