Don't use hasAttribute in SVGFilterPrimitiveStandardAttributes

Don't use hasAttribute in SVGFilterPrimitiveStandardAttributes

Use isSpecified() to query for existance in setStandardAttributes. This
avoids triggering unnecessary attribute synchronizations.

While here, drop the "double precondition" checking since the only caller
of this function checks this already.

BUG=235256
Committed: https://crrev.com/bfcc836db3fdba3ac761b1fe41c3347e75c31f94
Cr-Commit-Position: refs/heads/master@{#397202}

[fs, 2016-06-01 17:26:12]

Description was changed from

==========
Don't use hasAttribute in SVGFilterPrimitiveStandardAttributes

Use isSpecified() to query for existance in setStandardAttributes. This
avoids triggering unnecessary attribute synchronizations.

BUG=235256
==========

to

==========
Don't use hasAttribute in SVGFilterPrimitiveStandardAttributes

Use isSpecified() to query for existance in setStandardAttributes. This
avoids triggering unnecessary attribute synchronizations.

While here, drop the "double precondition" checking since the only caller
of this function checks this already.

BUG=235256
==========

[fs, 2016-06-01 18:26:43]

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

[fs, 2016-06-01 18:26:44]

Looked like this was the only mole left in this bug. Whack it.

[pdr., 2016-06-01 19:13:40]

Nice nice nice!

LGTM

[fs, 2016-06-01 19:16:02]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-06-01 19:16:41]

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

[commit-bot: I haz the power, 2016-06-01 19:22:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-01 19:23:44]

Description was changed from

==========
Don't use hasAttribute in SVGFilterPrimitiveStandardAttributes

Use isSpecified() to query for existance in setStandardAttributes. This
avoids triggering unnecessary attribute synchronizations.

While here, drop the "double precondition" checking since the only caller
of this function checks this already.

BUG=235256
==========

to

==========
Don't use hasAttribute in SVGFilterPrimitiveStandardAttributes

Use isSpecified() to query for existance in setStandardAttributes. This
avoids triggering unnecessary attribute synchronizations.

While here, drop the "double precondition" checking since the only caller
of this function checks this already.

BUG=235256

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

[commit-bot: I haz the power, 2016-06-01 19:23:45]

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