Use references some more in SVGUseElement

Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

BUG=589682
Committed: https://crrev.com/ee36663a57409b66d72776c462f00427b128e3b4
Cr-Commit-Position: refs/heads/master@{#379031}

[fs, 2016-03-03 13:07:35]

Description was changed from

==========
Use references where possible in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

For transferUseWidthAndHeightIfNeeded, the third parameter is removed,
because it can always be inferred from the second parameter.
The ASSERT that was previously only residing in svgAttributeChanged, is
moved into the function itself.

BUG=589682
==========

to

==========
Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

For transferUseWidthAndHeightIfNeeded, the third parameter is removed,
because it can always be inferred from the second parameter.
The ASSERT that was previously only residing in svgAttributeChanged, is
moved into the function itself.

BUG=589682
==========

[fs, 2016-03-03 13:27:55]

Description was changed from

==========
Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

For transferUseWidthAndHeightIfNeeded, the third parameter is removed,
because it can always be inferred from the second parameter.
The ASSERT that was previously only residing in svgAttributeChanged, is
moved into the function itself.

BUG=589682
==========

to

==========
Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

For transferUseWidthAndHeightIfNeeded, the third parameter is removed,
because it can always be inferred from the second parameter.

BUG=589682
==========

[fs, 2016-03-03 14:31:31]

Description was changed from

==========
Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

For transferUseWidthAndHeightIfNeeded, the third parameter is removed,
because it can always be inferred from the second parameter.

BUG=589682
==========

to

==========
Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

BUG=589682
==========

[fs, 2016-03-03 14:31:43]

Patchset #1 (id:1) has been deleted

[fs, 2016-03-03 14:31:50]

Patchset #1 (id:20001) has been deleted

[fs, 2016-03-03 15:26:22]

fs@opera.com changed reviewers:
+ pdr@chromium.org, schenney@chromium.org

[Stephen Chennney, 2016-03-03 16:27:36]

LGTM. Thanks for the ongoing work. Trybot failures are unrelated.

[fs, 2016-03-03 16:31:21]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-03-03 16:31:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1762633002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1762633002/40001

[commit-bot: I haz the power, 2016-03-03 17:38:02]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-03-03 17:39:16]

Description was changed from

==========
Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

BUG=589682
==========

to

==========
Use references some more in SVGUseElement

Mostly changes to pass SVGElement& rather than SVGElement* - and some
cleanup/removal of dead null-checks etc. as a consequence.

BUG=589682

Committed: https://crrev.com/ee36663a57409b66d72776c462f00427b128e3b4
Cr-Commit-Position: refs/heads/master@{#379031}
==========

[commit-bot: I haz the power, 2016-03-03 17:39:17]

Patchset 1 (id:??) landed as
https://crrev.com/ee36663a57409b66d72776c462f00427b128e3b4
Cr-Commit-Position: refs/heads/master@{#379031}