Round of CSS loader cleanup.
Categories
(Core :: CSS Parsing and Computation, task, P3)
Tracking
()
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(18 files, 1 obsolete file)
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review |
Assignee | ||
Comment 1•6 years ago
|
||
I'm going to tweak it a bit.
Assignee | ||
Comment 2•6 years ago
|
||
Little pet peeve o' mine.
Depends on D40687
Assignee | ||
Comment 3•6 years ago
|
||
Depends on D40688
Assignee | ||
Comment 4•6 years ago
|
||
Now that Sheets is out of line this can move to the cpp file too.
Depends on D40689
Assignee | ||
Comment 5•6 years ago
|
||
Rather than checking after-the fact and dropping the cached result on the floor.
Depends on D40690
Assignee | ||
Comment 6•6 years ago
|
||
Just moving code around but hopefully the new code is nicer :)
Depends on D40691
Assignee | ||
Comment 7•6 years ago
|
||
Also, it's infallible, so no need to have an error state.
Depends on D40692
Updated•6 years ago
|
Assignee | ||
Comment 8•6 years ago
|
||
It doesn't do much for them.
Assignee | ||
Comment 9•6 years ago
|
||
Assignee | ||
Comment 10•6 years ago
|
||
Assignee | ||
Comment 11•6 years ago
|
||
On the fence on this one, but I do think it's nicer.
Assignee | ||
Comment 12•6 years ago
|
||
They're bad, specially if they do vastly different thing in overloaded
functions, like aUseSystemPrincipal and aIsPreload. :)
Assignee | ||
Comment 13•6 years ago
|
||
It's a refptr hashtable, really.
Assignee | ||
Comment 14•6 years ago
|
||
Assignee | ||
Comment 15•6 years ago
|
||
Saving a refcount bump is not worth the churn. Use a proper RefPtr<>
everywhere instead of manual refcounting, and don't make DoSheetComplete call
NS_RELEASE unconditionally.
Also, make clear by using references where null is expected or not.
Also, properly use a RefPtr in mPendingDatas, since they are strong pointers,
in fact.
Finally, remove some unused macros from nsCSSValue of which this code was the
last consumer.
Assignee | ||
Comment 16•6 years ago
|
||
This is mostly straight-forward cleanup, but there's a behavior change which
was an oversight from bug 1386840, the regular mObservers stuff didn't pass the
right thing (was passing only mWasAlternate rather than whether it was
deferred).
I think that as a result, only in XML documents (since nsXMLContentSink is the
only thing that adds itself as a global CSS loader observer that ever looks at
this boolean), we may end up breaking this assert:
(If there are any sheets with non-matching media queries and such).
But I couldn't find a test-case that broke it (tried SVG / XHTML), and in other
documents like pure XML you cannot specify a media query...
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Comment 17•6 years ago
|
||
Comment 18•6 years ago
|
||
bugherder |
Assignee | ||
Comment 19•6 years ago
|
||
Assignee | ||
Comment 20•6 years ago
|
||
Assignee | ||
Comment 21•6 years ago
|
||
Comment 22•6 years ago
|
||
Updated•6 years ago
|
Comment 23•6 years ago
|
||
Updated•6 years ago
|
Updated•6 years ago
|
Comment 24•6 years ago
|
||
Comment 25•6 years ago
|
||
Updated•6 years ago
|
Updated•6 years ago
|
Comment 26•6 years ago
|
||
bugherder |
Comment 27•6 years ago
|
||
Thanks for this cleanup!
Comment 28•6 years ago
|
||
Comment 29•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4918a9dc15d7
https://hg.mozilla.org/mozilla-central/rev/eec457de4550
https://hg.mozilla.org/mozilla-central/rev/cf17f0433abf
https://hg.mozilla.org/mozilla-central/rev/db8dfad2e3b3
https://hg.mozilla.org/mozilla-central/rev/589addc490dc
https://hg.mozilla.org/mozilla-central/rev/1f4c64e73cfc
https://hg.mozilla.org/mozilla-central/rev/aec66f88bfd0
https://hg.mozilla.org/mozilla-central/rev/a28efee19e9c
https://hg.mozilla.org/mozilla-central/rev/3d9de500e9c5
https://hg.mozilla.org/mozilla-central/rev/e10941ec60df
https://hg.mozilla.org/mozilla-central/rev/852c5aaa34f7
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Description
•