content.css can be loaded only for web content

RESOLVED INVALID

Status

RESOLVED INVALID
9 years ago
8 years ago

People

(Reporter: vingtetun, Unassigned)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 441806 [details] [diff] [review]
Platform patch

Should I ask a review to Boris or someone else?
Comment on attachment 441804 [details] [diff] [review]
Patch

Why exactly do we want this patch? The BrowserStyleSheets component allows us
to apply the web theme to _any_ browser created in Fennec, not just our own.
One example is the reftest add-on.
(In reply to comment #2)
> (From update of attachment 441804 [details] [diff] [review])
> Why exactly do we want this patch?

My plan was to clean up css to make it faster by removing the need to check "html *" each time. (I've read to much of https://developer.mozilla.org/en/Writing_Efficient_CSS)

> The BrowserStyleSheets component allows us
> to apply the web theme to _any_ browser created in Fennec, not just our own.
> One example is the reftest add-on.

Hmmm, I've forgot that.

We could probably get the same result with a few tweaks in this file after all. I think i'll open another bug for the platform patch.
Comment on attachment 441806 [details] [diff] [review]
Platform patch

I've upload the platform patch to bug 403266.
Attachment #441806 - Attachment is obsolete: true
Comment on attachment 441804 [details] [diff] [review]
Patch

If you want to make some tweaks, that's OK, but I want to keep the component.
Attachment #441804 - Flags: review?(mark.finkle) → review-
I think this bug isn't needed anymore, right? If it is, REOPEN
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → INVALID
(In reply to comment #6)
> I think this bug isn't needed anymore, right? If it is, REOPEN

thanks for closing it!
You need to log in before you can comment on or make changes to this bug.