Reduced the callers to the non-const version of CSSValueList::item()

Reduced the callers to the non-const version of CSSValueList::item()

Reduced the callers to the non-const version of CSSValueList::item().
All non-trivial callsites were changed to use the const version of the
item() method. This is pre-work to remove the non-const version
altogether.

Remaining callsites either require CSSValueList::append() to take a
const pointer (which would require CSSValueList to actually store
Member<const CSSValue>s), or require some other cascading const change.

NOTE: This patch also involved making CSSFontFaceSrcValue::fetch() a
const method, and making the cached member m_fetched mutable.

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

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

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

[Timothy Loh, 2016-06-03 05:35:51]

lgtm

[sashab, 2016-06-06 05:36:29]

The CQ bit was checked by sashab@chromium.org

[sashab, 2016-06-06 05:36:30]

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/2036943002/#ps20001
(title: "Rebase")

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

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

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

Committed patchset #2 (id:20001)

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

Description was changed from

==========
Reduced the callers to the non-const version of CSSValueList::item()

Reduced the callers to the non-const version of CSSValueList::item().
All non-trivial callsites were changed to use the const version of the
item() method. This is pre-work to remove the non-const version
altogether.

Remaining callsites either require CSSValueList::append() to take a
const pointer (which would require CSSValueList to actually store
Member<const CSSValue>s), or require some other cascading const change.

NOTE: This patch also involved making CSSFontFaceSrcValue::fetch() a
const method, and making the cached member m_fetched mutable.

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

to

==========
Reduced the callers to the non-const version of CSSValueList::item()

Reduced the callers to the non-const version of CSSValueList::item().
All non-trivial callsites were changed to use the const version of the
item() method. This is pre-work to remove the non-const version
altogether.

Remaining callsites either require CSSValueList::append() to take a
const pointer (which would require CSSValueList to actually store
Member<const CSSValue>s), or require some other cascading const change.

NOTE: This patch also involved making CSSFontFaceSrcValue::fetch() a
const method, and making the cached member m_fetched mutable.

BUG=526586

Committed: https://crrev.com/4c39bbe8d34bb44f9f1b484e3c98a0c1b4ae5f6c
Cr-Commit-Position: refs/heads/master@{#397979}
==========

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

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