Closed
Bug 1345698
Opened 7 years ago
Closed 7 years ago
stylo: Implement access to CSSNamespaceRule
Categories
(Core :: CSS Parsing and Computation, enhancement, P2)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla55
| Tracking | Status | |
|---|---|---|
| firefox55 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
References
Details
Attachments
(5 files)
|
59 bytes,
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
59 bytes,
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
59 bytes,
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
59 bytes,
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
59 bytes,
text/x-review-board-request
|
heycam
:
review+
|
Details |
No description provided.
|
Assignee | |
Comment 1•7 years ago
|
||
I'll work on this after bug 1315601. Since this is a trivial one to implement, I'd like to use this bug to do some code simplification as well.
Assignee: nobody → xidorn+moz
Depends on: 1315601
|
||
Updated•7 years ago
|
Priority: -- → P2
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Comment 7•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847031 [details] Bug 1345698 part 1 - Unify common code for implementing a rule type. https://reviewboard.mozilla.org/r/120074/#review122284
Attachment #8847031 -
Flags: review?(cam) → review+
|
|
||
Comment 8•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847032 [details] Bug 1345698 part 2 - Add bindings for namespace rule. https://reviewboard.mozilla.org/r/120076/#review122290
Attachment #8847032 -
Flags: review?(cam) → review+
|
|
||
Comment 9•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847033 [details] Bug 1345698 part 3 - Fix a build failure for adding source. https://reviewboard.mozilla.org/r/120078/#review122292
Attachment #8847033 -
Flags: review?(cam) → review+
|
|
||
Comment 10•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847034 [details] Bug 1345698 part 4 - Create a new CSSNamespaceRule as a base class. https://reviewboard.mozilla.org/r/120080/#review122298
Attachment #8847034 -
Flags: review?(cam) → review+
|
|
||
Comment 11•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847035 [details] Bug 1345698 part 5 - Add support of CSSNamespaceRule for stylo. https://reviewboard.mozilla.org/r/120082/#review122304
Attachment #8847035 -
Flags: review?(cam) → review+
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Comment 17•7 years ago
|
||
Pushed by xquan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/52ba7a56ee83 part 1 - Unify common code for implementing a rule type. r=heycam https://hg.mozilla.org/integration/autoland/rev/7caba57b2d1b part 2 - Add bindings for namespace rule. r=heycam https://hg.mozilla.org/integration/autoland/rev/fb56873eb923 part 3 - Fix a build failure for adding source. r=heycam https://hg.mozilla.org/integration/autoland/rev/0ee93f46f578 part 4 - Create a new CSSNamespaceRule as a base class. r=heycam https://hg.mozilla.org/integration/autoland/rev/3123f116d6dc part 5 - Add support of CSSNamespaceRule for stylo. r=heycam
|
||
Comment 18•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/52ba7a56ee83 https://hg.mozilla.org/mozilla-central/rev/7caba57b2d1b https://hg.mozilla.org/mozilla-central/rev/fb56873eb923 https://hg.mozilla.org/mozilla-central/rev/0ee93f46f578 https://hg.mozilla.org/mozilla-central/rev/3123f116d6dc
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•