Closed Bug 1516194 Opened 5 years ago Closed 5 years ago

Move nsSMILNullType into the mozilla namespace

Categories

(Core :: SVG, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla66
Tracking Status
firefox66 --- fixed

People

(Reporter: longsonr, Assigned: longsonr)

Details

Attachments

(2 files)

      No description provided.
Attached patch patchSplinter Review
Assignee: nobody → longsonr
Attachment #9033135 - Flags: review?(dholbert)
Comment on attachment 9033135 [details] [diff] [review]
patch

Review of attachment 9033135 [details] [diff] [review]:
-----------------------------------------------------------------

r=me

(Looks like this will need minor unbitrotting for the SMILLFloatType changes in bug 1516175.)
Attachment #9033135 - Flags: review?(dholbert) → review+
Pushed by longsonr@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3a5eb424e655
rename nsSMILNullType to SMILNullType and move it to the mozilla namespace r=dholbert
Looks like some reformatting got missed from the initial patch.
Attachment #9033420 - Flags: review?(dholbert)
Comment on attachment 9033420 [details] [diff] [review]
Part 2 - fix indentation

has r+ in bug 1516521
Attachment #9033420 - Flags: review?(dholbert) → review+
Pushed by longsonr@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/006f69326ccc
fix identation in SMILNullType.cpp r=dholbert
Thanks!
https://hg.mozilla.org/mozilla-central/rev/3a5eb424e655
https://hg.mozilla.org/mozilla-central/rev/006f69326ccc
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: