Closed
Bug 1264837
Opened 9 years ago
Closed 9 years ago
Remove the parent class FooSuper typedef convention in layout/
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla48
| Tracking | Status | |
|---|---|---|
| firefox48 | --- | fixed |
People
(Reporter: TYLin, Assigned: TYLin)
References
Details
Attachments
(45 files)
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
|
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
Filed per bug 1260090 comment 10 by dbaron. We'd better have the consistency for getting rid of the parent class FooSuper typedef for all classes in layout.
|
Assignee | |
Updated•9 years ago
|
|
|
Assignee | |
Comment 1•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/46995/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/46995/
Attachment #8742235 -
Flags: review?(dholbert)
Attachment #8742236 -
Flags: review?(dholbert)
Attachment #8742237 -
Flags: review?(dholbert)
Attachment #8742238 -
Flags: review?(dholbert)
Attachment #8742239 -
Flags: review?(dholbert)
Attachment #8742240 -
Flags: review?(dholbert)
Attachment #8742241 -
Flags: review?(dholbert)
Attachment #8742242 -
Flags: review?(dholbert)
Attachment #8742243 -
Flags: review?(dholbert)
Attachment #8742244 -
Flags: review?(dholbert)
Attachment #8742245 -
Flags: review?(dholbert)
Attachment #8742246 -
Flags: review?(dholbert)
Attachment #8742247 -
Flags: review?(dholbert)
Attachment #8742248 -
Flags: review?(dholbert)
Attachment #8742249 -
Flags: review?(dholbert)
Attachment #8742250 -
Flags: review?(dholbert)
Attachment #8742251 -
Flags: review?(dholbert)
Attachment #8742252 -
Flags: review?(dholbert)
Attachment #8742253 -
Flags: review?(dholbert)
Attachment #8742254 -
Flags: review?(dholbert)
Attachment #8742255 -
Flags: review?(dholbert)
Attachment #8742256 -
Flags: review?(dholbert)
Attachment #8742257 -
Flags: review?(dholbert)
Attachment #8742258 -
Flags: review?(dholbert)
Attachment #8742259 -
Flags: review?(dholbert)
Attachment #8742260 -
Flags: review?(dholbert)
Attachment #8742261 -
Flags: review?(dholbert)
Attachment #8742262 -
Flags: review?(dholbert)
Attachment #8742263 -
Flags: review?(dholbert)
Attachment #8742264 -
Flags: review?(dholbert)
Attachment #8742265 -
Flags: review?(dholbert)
Attachment #8742266 -
Flags: review?(dholbert)
Attachment #8742267 -
Flags: review?(dholbert)
Attachment #8742268 -
Flags: review?(dholbert)
Attachment #8742269 -
Flags: review?(dholbert)
Attachment #8742270 -
Flags: review?(dholbert)
Attachment #8742271 -
Flags: review?(dholbert)
Attachment #8742272 -
Flags: review?(dholbert)
Attachment #8742273 -
Flags: review?(dholbert)
Attachment #8742274 -
Flags: review?(dholbert)
Attachment #8742275 -
Flags: review?(dholbert)
Attachment #8742276 -
Flags: review?(dholbert)
Attachment #8742277 -
Flags: review?(dholbert)
Attachment #8742278 -
Flags: review?(dholbert)
Attachment #8742279 -
Flags: review?(dholbert)
|
|
Assignee | |
Comment 2•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/46997/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/46997/
|
|
Assignee | |
Comment 3•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/46999/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/46999/
|
|
Assignee | |
Comment 4•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47001/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47001/
|
|
Assignee | |
Comment 5•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47003/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47003/
|
|
Assignee | |
Comment 6•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47005/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47005/
|
|
Assignee | |
Comment 7•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47007/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47007/
|
|
Assignee | |
Comment 8•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47009/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47009/
|
|
Assignee | |
Comment 9•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47011/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47011/
|
|
Assignee | |
Comment 10•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47013/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47013/
|
|
Assignee | |
Comment 11•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47015/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47015/
|
|
Assignee | |
Comment 12•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47017/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47017/
|
|
Assignee | |
Comment 13•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47019/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47019/
|
|
Assignee | |
Comment 14•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47021/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47021/
|
|
Assignee | |
Comment 15•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47023/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47023/
|
|
Assignee | |
Comment 16•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47025/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47025/
|
|
Assignee | |
Comment 17•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47027/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47027/
|
|
Assignee | |
Comment 18•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47029/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47029/
|
|
Assignee | |
Comment 19•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47031/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47031/
|
|
Assignee | |
Comment 20•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47033/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47033/
|
|
Assignee | |
Comment 21•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47035/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47035/
|
|
Assignee | |
Comment 22•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47037/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47037/
|
|
Assignee | |
Comment 23•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47039/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47039/
|
|
Assignee | |
Comment 24•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47041/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47041/
|
|
Assignee | |
Comment 25•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47043/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47043/
|
|
Assignee | |
Comment 26•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47045/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47045/
|
|
Assignee | |
Comment 27•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47047/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47047/
|
|
Assignee | |
Comment 28•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47049/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47049/
|
|
Assignee | |
Comment 29•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47051/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47051/
|
|
Assignee | |
Comment 30•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47053/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47053/
|
|
Assignee | |
Comment 31•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47055/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47055/
|
|
Assignee | |
Comment 32•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47057/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47057/
|
|
Assignee | |
Comment 33•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47059/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47059/
|
|
Assignee | |
Comment 34•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47061/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47061/
|
|
Assignee | |
Comment 35•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47063/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47063/
|
|
Assignee | |
Comment 36•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47065/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47065/
|
|
Assignee | |
Comment 37•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47067/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47067/
|
|
Assignee | |
Comment 38•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47069/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47069/
|
|
Assignee | |
Comment 39•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47071/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47071/
|
|
Assignee | |
Comment 40•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47073/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47073/
|
|
Assignee | |
Comment 41•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47075/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47075/
|
|
Assignee | |
Comment 42•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47077/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47077/
|
|
Assignee | |
Comment 43•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47079/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47079/
|
|
Assignee | |
Comment 44•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47081/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47081/
|
|
Assignee | |
Comment 45•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47083/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47083/
|
||
Updated•9 years ago
|
Attachment #8742235 -
Flags: review?(dholbert) → review+
|
|
||
Comment 46•9 years ago
|
||
Comment on attachment 8742235 [details]
MozReview Request: Bug 1264837 Part 1 - Remove nsTextBoxFrameSuper.
https://reviewboard.mozilla.org/r/46995/#review43723
|
|
||
Comment 47•9 years ago
|
||
Comment on attachment 8742236 [details]
MozReview Request: Bug 1264837 Part 2 - Remove nsColorControlFrameSuper.
https://reviewboard.mozilla.org/r/46997/#review43729
Attachment #8742236 -
Flags: review?(dholbert) → review+
|
|
||
Comment 48•9 years ago
|
||
Comment on attachment 8742237 [details]
MozReview Request: Bug 1264837 Part 3 - Remove nsFormControlFrameSuper.
https://reviewboard.mozilla.org/r/46999/#review43731
Attachment #8742237 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742238 -
Flags: review?(dholbert) → review+
|
|
||
Comment 49•9 years ago
|
||
Comment on attachment 8742238 [details]
MozReview Request: Bug 1264837 Part 4 - Remove nsImageControlFrameSuper.
https://reviewboard.mozilla.org/r/47001/#review43733
|
|
||
Comment 50•9 years ago
|
||
Comment on attachment 8742239 [details]
MozReview Request: Bug 1264837 Part 5 - Remove nsFlexContainerFrameSuper.
https://reviewboard.mozilla.org/r/47003/#review43735
Attachment #8742239 -
Flags: review?(dholbert) → review+
|
|
||
Comment 51•9 years ago
|
||
Comment on attachment 8742240 [details]
MozReview Request: Bug 1264837 Part 6 - Remove ImageFrameSuper.
https://reviewboard.mozilla.org/r/47005/#review43739
Attachment #8742240 -
Flags: review?(dholbert) → review+
|
|
||
Comment 52•9 years ago
|
||
Comment on attachment 8742241 [details]
MozReview Request: Bug 1264837 Part 7 - Remove nsInlineFrameBase
https://reviewboard.mozilla.org/r/47007/#review43741
Attachment #8742241 -
Flags: review?(dholbert) → review+
|
|
||
Comment 53•9 years ago
|
||
Comment on attachment 8742242 [details]
MozReview Request: Bug 1264837 Part 8 - Remove nsPluginFrameSuper.
https://reviewboard.mozilla.org/r/47009/#review43757
Attachment #8742242 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742243 -
Flags: review?(dholbert) → review+
|
|
||
Comment 54•9 years ago
|
||
Comment on attachment 8742243 [details]
MozReview Request: Bug 1264837 Part 9 - Remove nsRubyBaseFrameSuper.
https://reviewboard.mozilla.org/r/47011/#review43761
|
|
||
Updated•9 years ago
|
Attachment #8742244 -
Flags: review?(dholbert) → review+
|
|
||
Comment 55•9 years ago
|
||
Comment on attachment 8742244 [details]
MozReview Request: Bug 1264837 Part 10 - Remove nsRubyContentFrameSuper.
https://reviewboard.mozilla.org/r/47013/#review43763
|
|
||
Comment 56•9 years ago
|
||
Comment on attachment 8742245 [details]
MozReview Request: Bug 1264837 Part 11 - Remove nsRubyFrameSuper.
https://reviewboard.mozilla.org/r/47015/#review43765
Attachment #8742245 -
Flags: review?(dholbert) → review+
|
|
||
Comment 57•9 years ago
|
||
Comment on attachment 8742246 [details]
MozReview Request: Bug 1264837 Part 12 - Remove nsRubyTextContainerFrameSuper.
https://reviewboard.mozilla.org/r/47017/#review43773
Attachment #8742246 -
Flags: review?(dholbert) → review+
|
|
||
Comment 58•9 years ago
|
||
Comment on attachment 8742247 [details]
MozReview Request: Bug 1264837 Part 13 - Remove nsRubyTextFrameSuper.
https://reviewboard.mozilla.org/r/47019/#review43777
Attachment #8742247 -
Flags: review?(dholbert) → review+
|
|
||
Comment 59•9 years ago
|
||
Comment on attachment 8742248 [details]
MozReview Request: Bug 1264837 Part 14 - Remove nsSubDocumentFrameSuper.
https://reviewboard.mozilla.org/r/47021/#review43779
Attachment #8742248 -
Flags: review?(dholbert) → review+
|
|
||
Comment 60•9 years ago
|
||
Comment on attachment 8742249 [details]
MozReview Request: Bug 1264837 Part 15 - Remove nsVideoFrameBase
https://reviewboard.mozilla.org/r/47023/#review43789
Attachment #8742249 -
Flags: review?(dholbert) → review+
|
|
||
Comment 61•9 years ago
|
||
Comment on attachment 8742250 [details]
MozReview Request: Bug 1264837 Part 16 - Remove ViewportFrame::Super.
https://reviewboard.mozilla.org/r/47025/#review43793
Attachment #8742250 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742251 -
Flags: review?(dholbert) → review+
|
|
||
Comment 62•9 years ago
|
||
Comment on attachment 8742251 [details]
MozReview Request: Bug 1264837 Part 17 - Remove nsSVGAFrameBase.
https://reviewboard.mozilla.org/r/47027/#review43795
|
|
||
Comment 63•9 years ago
|
||
Comment on attachment 8742252 [details]
MozReview Request: Bug 1264837 Part 18 - Remove nsSVGClipPathFrameBase.
https://reviewboard.mozilla.org/r/47029/#review43797
Attachment #8742252 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742253 -
Flags: review?(dholbert) → review+
|
|
||
Comment 64•9 years ago
|
||
Comment on attachment 8742253 [details]
MozReview Request: Bug 1264837 Part 19 - Remove nsSVGContainerFrameBase.
https://reviewboard.mozilla.org/r/47031/#review43799
|
|
||
Comment 65•9 years ago
|
||
Comment on attachment 8742254 [details]
MozReview Request: Bug 1264837 Part 20 - Remove nsSVGFilterFrameBase.
https://reviewboard.mozilla.org/r/47033/#review43803
Attachment #8742254 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742255 -
Flags: review?(dholbert) → review+
|
|
||
Comment 66•9 years ago
|
||
Comment on attachment 8742255 [details]
MozReview Request: Bug 1264837 Part 21 - Remove nsSVGForeignObjectFrameBase.
https://reviewboard.mozilla.org/r/47035/#review43809
Ship It!
|
|
||
Comment 67•9 years ago
|
||
Comment on attachment 8742256 [details]
MozReview Request: Bug 1264837 Part 22 - Remove nsSVGGenericContainerFrameBase.
https://reviewboard.mozilla.org/r/47037/#review43811
Ship It!
Attachment #8742256 -
Flags: review?(dholbert) → review+
|
|
||
Comment 68•9 years ago
|
||
Comment on attachment 8742257 [details]
MozReview Request: Bug 1264837 Part 23 - Remove nsSVGGFrameBase.
https://reviewboard.mozilla.org/r/47039/#review43813
Ship It!
Attachment #8742257 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742258 -
Flags: review?(dholbert) → review+
|
|
||
Comment 69•9 years ago
|
||
Comment on attachment 8742258 [details]
MozReview Request: Bug 1264837 Part 24 - Remove nsSVGGradientFrameBase.
https://reviewboard.mozilla.org/r/47041/#review43817
Ship It!
|
|
||
Updated•9 years ago
|
Attachment #8742259 -
Flags: review?(dholbert) → review+
|
|
||
Comment 70•9 years ago
|
||
Comment on attachment 8742259 [details]
MozReview Request: Bug 1264837 Part 25 - Remove nsSVGLinearGradientFrameBase.
https://reviewboard.mozilla.org/r/47043/#review43823
Ship It!
|
|
||
Comment 71•9 years ago
|
||
Comment on attachment 8742260 [details]
MozReview Request: Bug 1264837 Part 26 - Remove nsSVGRadialGradientFrameBase.
https://reviewboard.mozilla.org/r/47045/#review43829
Ship It!
Attachment #8742260 -
Flags: review?(dholbert) → review+
|
|
||
Comment 72•9 years ago
|
||
Comment on attachment 8742261 [details]
MozReview Request: Bug 1264837 Part 27 - Remove nsSVGImageFrameBase.
https://reviewboard.mozilla.org/r/47047/#review43831
Ship It!
Attachment #8742261 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742262 -
Flags: review?(dholbert) → review+
|
|
||
Comment 73•9 years ago
|
||
Comment on attachment 8742262 [details]
MozReview Request: Bug 1264837 Part 28 - Remove nsSVGInnerSVGFrameBase.
https://reviewboard.mozilla.org/r/47049/#review43833
Ship It!
|
|
||
Comment 74•9 years ago
|
||
Comment on attachment 8742263 [details]
MozReview Request: Bug 1264837 Part 29 - Remove nsSVGMarkerFrameBase.
https://reviewboard.mozilla.org/r/47051/#review43835
Ship It!
Attachment #8742263 -
Flags: review?(dholbert) → review+
|
|
||
Comment 75•9 years ago
|
||
Comment on attachment 8742264 [details]
MozReview Request: Bug 1264837 Part 30 - Remove nsSVGMarkerAnonChildFrameBase.
https://reviewboard.mozilla.org/r/47053/#review43837
Ship It!
Attachment #8742264 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742265 -
Flags: review?(dholbert) → review+
|
|
||
Comment 76•9 years ago
|
||
Comment on attachment 8742265 [details]
MozReview Request: Bug 1264837 Part 31 - Remove nsSVGMaskFrameBase.
https://reviewboard.mozilla.org/r/47055/#review43839
Ship It!
|
|
||
Updated•9 years ago
|
Attachment #8742266 -
Flags: review?(dholbert) → review+
|
|
||
Comment 77•9 years ago
|
||
Comment on attachment 8742266 [details]
MozReview Request: Bug 1264837 Part 32 - Remove nsSVGOuterSVGFrameBase.
https://reviewboard.mozilla.org/r/47057/#review43841
Ship It!
|
|
||
Comment 78•9 years ago
|
||
Comment on attachment 8742267 [details]
MozReview Request: Bug 1264837 Part 33 - Remove nsSVGOuterSVGAnonChildFrameBase.
https://reviewboard.mozilla.org/r/47059/#review43845
Ship It!
Attachment #8742267 -
Flags: review?(dholbert) → review+
|
|
||
Comment 79•9 years ago
|
||
Comment on attachment 8742268 [details]
MozReview Request: Bug 1264837 Part 34 - Remove nsSVGPaintServerFrameBase.
https://reviewboard.mozilla.org/r/47061/#review43851
Ship It!
Attachment #8742268 -
Flags: review?(dholbert) → review+
|
|
||
Comment 80•9 years ago
|
||
Comment on attachment 8742269 [details]
MozReview Request: Bug 1264837 Part 35 - Remove nsSVGPathGeometryFrameBase.
https://reviewboard.mozilla.org/r/47063/#review43853
Ship It!
Attachment #8742269 -
Flags: review?(dholbert) → review+
|
|
||
Comment 81•9 years ago
|
||
Comment on attachment 8742270 [details]
MozReview Request: Bug 1264837 Part 36 - Remove nsSVGPatternFrameBase.
https://reviewboard.mozilla.org/r/47065/#review43855
Ship It!
Attachment #8742270 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742271 -
Flags: review?(dholbert) → review+
|
|
||
Comment 82•9 years ago
|
||
Comment on attachment 8742271 [details]
MozReview Request: Bug 1264837 Part 37 - Remove nsSVGStopFrameBase.
https://reviewboard.mozilla.org/r/47067/#review43857
Ship It!
|
|
||
Comment 83•9 years ago
|
||
Comment on attachment 8742272 [details]
MozReview Request: Bug 1264837 Part 38 - Remove nsSVGSwitchFrameBase.
https://reviewboard.mozilla.org/r/47069/#review43859
Ship It!
Attachment #8742272 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742273 -
Flags: review?(dholbert) → review+
|
|
||
Comment 84•9 years ago
|
||
Comment on attachment 8742273 [details]
MozReview Request: Bug 1264837 Part 39 - Remove nsSVGUseFrameBase.
https://reviewboard.mozilla.org/r/47071/#review43861
Ship It!
|
|
||
Comment 85•9 years ago
|
||
Comment on attachment 8742274 [details]
MozReview Request: Bug 1264837 Part 40 - Remove SVGFEContainerFrameBase.
https://reviewboard.mozilla.org/r/47073/#review43863
Ship It!
Attachment #8742274 -
Flags: review?(dholbert) → review+
|
|
||
Comment 86•9 years ago
|
||
Comment on attachment 8742275 [details]
MozReview Request: Bug 1264837 Part 41 - Remove SVGFEImageFrameBase.
https://reviewboard.mozilla.org/r/47075/#review43895
Ship It!
Attachment #8742275 -
Flags: review?(dholbert) → review+
|
|
||
Updated•9 years ago
|
Attachment #8742276 -
Flags: review?(dholbert) → review+
|
|
||
Comment 87•9 years ago
|
||
Comment on attachment 8742276 [details]
MozReview Request: Bug 1264837 Part 42 - Remove SVGFELeafFrameBase.
https://reviewboard.mozilla.org/r/47077/#review43897
Ship It!
|
|
||
Comment 88•9 years ago
|
||
Comment on attachment 8742277 [details]
MozReview Request: Bug 1264837 Part 43 - Remove SVGFEUnstyledLeafFrameBase.
https://reviewboard.mozilla.org/r/47079/#review43909
Ship It!
Attachment #8742277 -
Flags: review?(dholbert) → review+
|
|
||
Comment 89•9 years ago
|
||
Comment on attachment 8742278 [details]
MozReview Request: Bug 1264837 Part 44 - Remove SVGTextFrameBase.
https://reviewboard.mozilla.org/r/47081/#review43911
Ship It!
Attachment #8742278 -
Flags: review?(dholbert) → review+
|
|
||
Comment 90•9 years ago
|
||
Comment on attachment 8742279 [details]
MozReview Request: Bug 1264837 Part 45 - Remove SVGViewFrameBase.
https://reviewboard.mozilla.org/r/47083/#review43913
Ship It!
Attachment #8742279 -
Flags: review?(dholbert) → review+
|
|
||
Comment 91•9 years ago
|
||
Thank you for following through on this, so quickly & so thoroughly! Now let's get it landed, before someone bitrots you! :)
Flags: needinfo?(tlin)
|
||
Comment 92•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/abd6f4bc9ba7
https://hg.mozilla.org/integration/mozilla-inbound/rev/c857149cf48a
https://hg.mozilla.org/integration/mozilla-inbound/rev/7fd8b1e1be14
https://hg.mozilla.org/integration/mozilla-inbound/rev/02d2f37a90cc
https://hg.mozilla.org/integration/mozilla-inbound/rev/737b1eb73eb5
https://hg.mozilla.org/integration/mozilla-inbound/rev/d155d4cc4312
https://hg.mozilla.org/integration/mozilla-inbound/rev/ae3e61a94e81
https://hg.mozilla.org/integration/mozilla-inbound/rev/dedd67270907
https://hg.mozilla.org/integration/mozilla-inbound/rev/c1d823f092f3
https://hg.mozilla.org/integration/mozilla-inbound/rev/fab1dc151cdd
https://hg.mozilla.org/integration/mozilla-inbound/rev/1ea5e946e685
https://hg.mozilla.org/integration/mozilla-inbound/rev/f03a432e066c
https://hg.mozilla.org/integration/mozilla-inbound/rev/a44eef265b43
https://hg.mozilla.org/integration/mozilla-inbound/rev/85abd951afd1
https://hg.mozilla.org/integration/mozilla-inbound/rev/eb43adbaab69
https://hg.mozilla.org/integration/mozilla-inbound/rev/2f3123a99167
https://hg.mozilla.org/integration/mozilla-inbound/rev/d7b15866a041
https://hg.mozilla.org/integration/mozilla-inbound/rev/1a470a0d296e
https://hg.mozilla.org/integration/mozilla-inbound/rev/9b8fba24c222
https://hg.mozilla.org/integration/mozilla-inbound/rev/4d5e48e6c536
https://hg.mozilla.org/integration/mozilla-inbound/rev/47ab173e3a5a
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6c554476937
https://hg.mozilla.org/integration/mozilla-inbound/rev/a872c181ca9a
https://hg.mozilla.org/integration/mozilla-inbound/rev/bdc8feceeeb8
https://hg.mozilla.org/integration/mozilla-inbound/rev/079703139d43
https://hg.mozilla.org/integration/mozilla-inbound/rev/eb95ddffa2b0
https://hg.mozilla.org/integration/mozilla-inbound/rev/34f48a797e2a
https://hg.mozilla.org/integration/mozilla-inbound/rev/ebd26de8705e
https://hg.mozilla.org/integration/mozilla-inbound/rev/38df0143c983
https://hg.mozilla.org/integration/mozilla-inbound/rev/3e91b95e6fc0
https://hg.mozilla.org/integration/mozilla-inbound/rev/86b92343da55
https://hg.mozilla.org/integration/mozilla-inbound/rev/4b8f1454b8d7
https://hg.mozilla.org/integration/mozilla-inbound/rev/295dfad1d251
https://hg.mozilla.org/integration/mozilla-inbound/rev/7e8989ae11c6
https://hg.mozilla.org/integration/mozilla-inbound/rev/814ccf5291cb
https://hg.mozilla.org/integration/mozilla-inbound/rev/2806d27362bf
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a32e6070959
https://hg.mozilla.org/integration/mozilla-inbound/rev/8f7cc8ac46ea
https://hg.mozilla.org/integration/mozilla-inbound/rev/f923a29e9b72
https://hg.mozilla.org/integration/mozilla-inbound/rev/dd38967d24de
https://hg.mozilla.org/integration/mozilla-inbound/rev/b58454385cf9
https://hg.mozilla.org/integration/mozilla-inbound/rev/5d7bbe272e29
https://hg.mozilla.org/integration/mozilla-inbound/rev/d855f55ad32f
https://hg.mozilla.org/integration/mozilla-inbound/rev/7e962e85d091
https://hg.mozilla.org/integration/mozilla-inbound/rev/e995cbd2835c
|
||
Comment 94•9 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/abd6f4bc9ba7
https://hg.mozilla.org/mozilla-central/rev/c857149cf48a
https://hg.mozilla.org/mozilla-central/rev/7fd8b1e1be14
https://hg.mozilla.org/mozilla-central/rev/02d2f37a90cc
https://hg.mozilla.org/mozilla-central/rev/737b1eb73eb5
https://hg.mozilla.org/mozilla-central/rev/d155d4cc4312
https://hg.mozilla.org/mozilla-central/rev/ae3e61a94e81
https://hg.mozilla.org/mozilla-central/rev/dedd67270907
https://hg.mozilla.org/mozilla-central/rev/c1d823f092f3
https://hg.mozilla.org/mozilla-central/rev/fab1dc151cdd
https://hg.mozilla.org/mozilla-central/rev/1ea5e946e685
https://hg.mozilla.org/mozilla-central/rev/f03a432e066c
https://hg.mozilla.org/mozilla-central/rev/a44eef265b43
https://hg.mozilla.org/mozilla-central/rev/85abd951afd1
https://hg.mozilla.org/mozilla-central/rev/eb43adbaab69
https://hg.mozilla.org/mozilla-central/rev/2f3123a99167
https://hg.mozilla.org/mozilla-central/rev/d7b15866a041
https://hg.mozilla.org/mozilla-central/rev/1a470a0d296e
https://hg.mozilla.org/mozilla-central/rev/9b8fba24c222
https://hg.mozilla.org/mozilla-central/rev/4d5e48e6c536
https://hg.mozilla.org/mozilla-central/rev/47ab173e3a5a
https://hg.mozilla.org/mozilla-central/rev/e6c554476937
https://hg.mozilla.org/mozilla-central/rev/a872c181ca9a
https://hg.mozilla.org/mozilla-central/rev/bdc8feceeeb8
https://hg.mozilla.org/mozilla-central/rev/079703139d43
https://hg.mozilla.org/mozilla-central/rev/eb95ddffa2b0
https://hg.mozilla.org/mozilla-central/rev/34f48a797e2a
https://hg.mozilla.org/mozilla-central/rev/ebd26de8705e
https://hg.mozilla.org/mozilla-central/rev/38df0143c983
https://hg.mozilla.org/mozilla-central/rev/3e91b95e6fc0
https://hg.mozilla.org/mozilla-central/rev/86b92343da55
https://hg.mozilla.org/mozilla-central/rev/4b8f1454b8d7
https://hg.mozilla.org/mozilla-central/rev/295dfad1d251
https://hg.mozilla.org/mozilla-central/rev/7e8989ae11c6
https://hg.mozilla.org/mozilla-central/rev/814ccf5291cb
https://hg.mozilla.org/mozilla-central/rev/2806d27362bf
https://hg.mozilla.org/mozilla-central/rev/9a32e6070959
https://hg.mozilla.org/mozilla-central/rev/8f7cc8ac46ea
https://hg.mozilla.org/mozilla-central/rev/f923a29e9b72
https://hg.mozilla.org/mozilla-central/rev/dd38967d24de
https://hg.mozilla.org/mozilla-central/rev/b58454385cf9
https://hg.mozilla.org/mozilla-central/rev/5d7bbe272e29
https://hg.mozilla.org/mozilla-central/rev/d855f55ad32f
https://hg.mozilla.org/mozilla-central/rev/7e962e85d091
https://hg.mozilla.org/mozilla-central/rev/e995cbd2835c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•