The default bug view has changed. See this FAQ.

Rename nsDisplaySVG to nsDisplayOuterSVG in preparation for adding other SVG display list item types

RESOLVED FIXED in mozilla16

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

Trunk
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
This patch renames nsDisplaySVG to nsDisplayOuterSVG in preparation for adding other SVG display list item types in bug 641732.
(Assignee)

Comment 1

5 years ago
Created attachment 637497 [details] [diff] [review]
patch
Attachment #637497 - Flags: review?(longsonr)
Comment on attachment 637497 [details] [diff] [review]
patch

Stealing review.

>diff --git a/layout/svg/base/src/nsSVGOuterSVGFrame.cpp b/layout/svg/base/src/nsSVGOuterSVGFrame.cpp
>-class nsDisplaySVG : public nsDisplayItem {
>+class nsDisplayOuterSVG : public nsDisplayItem {
> public:
>-  nsDisplaySVG(nsDisplayListBuilder* aBuilder,
>+  nsDisplayOuterSVG(nsDisplayListBuilder* aBuilder,
>                nsSVGOuterSVGFrame* aFrame) :

That last line wants some indentation fixup.

Otherwise looks good -- r=me
Attachment #637497 - Flags: review?(longsonr) → review+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/d8308e2677f9
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/d8308e2677f9
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Updated

5 years ago
Blocks: 614732
You need to log in before you can comment on or make changes to this bug.