category name for mobile clh should start with 'z'

RESOLVED FIXED

Status

Fennec Graveyard
General
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: dougt, Assigned: dougt)

Tracking

Trunk
x86
Mac OS X

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
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

7 years ago
Created attachment 483080 [details] [diff] [review]
patch v.1
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+
I meant 'x'
(Assignee)

Comment 4

7 years ago
got it!

22:14 <@mfinkle> use "x"

http://hg.mozilla.org/mobile-browser/rev/2a0e63ecb5b2
(Assignee)

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 5

7 years ago
How can I verify this bug?
You need to log in before you can comment on or make changes to this bug.