Closed
Bug 404851
Opened 17 years ago
Closed 17 years ago
change "communicator/skin" to "global/skin"
Categories
(Other Applications :: DOM Inspector, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mfinkle, Assigned: mfinkle)
Details
Attachments
(1 file)
1.91 KB,
patch
|
sdwilsh
:
review+
neil
:
superreview+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
"chrome://communicator/skin" is being used to pull in the "chrome://global/skin" stylesheet. "communicator" was just importing "global" has been removed from toolkit. Switching explicitly to "global" shouldn't break anything as "global" has been around forever and will not break xulrunner apps that use DOMi.
Attachment #289719 -
Flags: review?(comrade693+bmo)
Comment 1•17 years ago
|
||
Comment on attachment 289719 [details] [diff] [review] switches to global skin r=sdwilsh
Attachment #289719 -
Flags: superreview?(neil)
Attachment #289719 -
Flags: review?(comrade693+bmo)
Attachment #289719 -
Flags: review+
Updated•17 years ago
|
Attachment #289719 -
Flags: superreview?(neil) → superreview+
Updated•17 years ago
|
Attachment #289719 -
Flags: approval1.9?
Comment 2•17 years ago
|
||
Comment on attachment 289719 [details] [diff] [review] switches to global skin a=beltzner on behalf of drivers
Attachment #289719 -
Flags: approval1.9? → approval1.9+
Comment 3•17 years ago
|
||
mfinkle - do you have checkin privileges?
Whiteboard: [has patch][has review][has sr][can land]
Updated•17 years ago
|
Assignee: nobody → mark.finkle
Assignee | ||
Comment 4•17 years ago
|
||
yep - checked into trunk
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [has patch][has review][has sr][can land]
You need to log in
before you can comment on or make changes to this bug.
Description
•