Change CSSValueList::removeAll and hasValue to take const CSSValue&

Change CSSValueList::removeAll and hasValue to take const CSSValue&

Change CSSValueList::removeAll and CSSValueList::hasValue to take const
references instead of pointers. This is more semantically correct and
safe, and is good pre-work for making CSSValueList::item() return const
references.

BUG=526586
Committed: https://crrev.com/d87f984cb32b4a55b95232b9c359269694cf8614
Cr-Commit-Position: refs/heads/master@{#397977}

[sashab, 2016-06-03 04:39:53]

sashab@chromium.org changed reviewers:
+ timloh@chromium.org

[Timothy Loh, 2016-06-03 04:57:42]

On 2016/06/03 04:39:59, sashab wrote:

lgtm

[sashab, 2016-06-03 04:59:13]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2016-06-03 04:59:36]

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

[sashab, 2016-06-03 06:22:04]

The CQ bit was unchecked by sashab@chromium.org

[sashab, 2016-06-06 00:59:52]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2016-06-06 01:00:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2033583006/1

[sashab, 2016-06-06 04:45:02]

The CQ bit was checked by sashab@chromium.org

[sashab, 2016-06-06 04:45:02]

The patchset sent to the CQ was uploaded after l-g-t-m from timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/2033583006/#ps20001
(title: "Rebase")

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

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

[commit-bot: I haz the power, 2016-06-06 05:35:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-06 05:36:17]

Description was changed from

==========
Change CSSValueList::removeAll and hasValue to take const CSSValue&

Change CSSValueList::removeAll and CSSValueList::hasValue to take const
references instead of pointers. This is more semantically correct and
safe, and is good pre-work for making CSSValueList::item() return const
references.

BUG=526586
==========

to

==========
Change CSSValueList::removeAll and hasValue to take const CSSValue&

Change CSSValueList::removeAll and CSSValueList::hasValue to take const
references instead of pointers. This is more semantically correct and
safe, and is good pre-work for making CSSValueList::item() return const
references.

BUG=526586

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

[commit-bot: I haz the power, 2016-06-06 05:36:18]

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