Closed Bug 404853 Opened 12 years ago Closed 12 years ago

change "communicator/skin" to "global/skin"

Categories

(Other Applications Graveyard :: Venkman JS Debugger, defect)

x86
Windows XP
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mfinkle, Assigned: mfinkle)

Details

Attachments

(1 file)

"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 Venkman.
Attachment #289720 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 289720 [details] [diff] [review]
switches to global skin

Looks like this works on just about everything. I still insist it's stupid that this communicator 'band-aid', as it was called yesterday, is being removed, but at least this patch works.

r=gijs
Attachment #289720 - Flags: review?(gijskruitbosch+bugs) → review+
Attachment #289720 - Flags: approval1.9?
Comment on attachment 289720 [details] [diff] [review]
switches to global skin

Venkman is NPOTB. You don't need a1.9 .
Attachment #289720 - Flags: approval1.9?
Assignee: rginda → mark.finkle
checked into trunk
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.