Rename consumeImage to consumeImageOrNone in CSSPropertyParser

Rename consumeImage to consumeImageOrNone in CSSPropertyParser

This patch renames consumeImage to consumeImageOrNone to more
accurately represent what the function does. It consumes an ident in
the case CSSValueNone is specified and consumes an image otherwise.

Committed: https://crrev.com/e1601140c77d33943d613e2242d48d9886f6ceb3
Cr-Commit-Position: refs/heads/master@{#383881}

[nainar, 2016-03-29 08:05:51]

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

[nainar, 2016-03-29 08:05:51]

PTAL?

Thanks!

[Timothy Loh, 2016-03-29 08:58:33]

lgtm

https://codereview.chromium.org/1842523003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp (right):

https://codereview.chromium.org/1842523003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp:2621: if
(range.peek().id() == CSSValueNone)
if you want you could remove this branch in this patch. or it'd be fine as a
tiny separate patch.

[nainar, 2016-03-29 22:39:52]

https://codereview.chromium.org/1842523003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp (right):

https://codereview.chromium.org/1842523003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp:2621: if
(range.peek().id() == CSSValueNone)
Removing as part of this patch. Thanks!

[nainar, 2016-03-29 22:39:54]

The CQ bit was checked by nainar@chromium.org

[nainar, 2016-03-29 22:39:54]

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

[commit-bot: I haz the power, 2016-03-29 22:40:18]

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

[commit-bot: I haz the power, 2016-03-30 00:56:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-30 00:58:03]

Description was changed from

==========
Rename consumeImage to consumeImageOrNone in CSSPropertyParser

This patch renames consumeImage to consumeImageOrNone to more
accurately represent what the function does. It consumes an ident in
the case CSSValueNone is specified and consumes an image otherwise.
==========

to

==========
Rename consumeImage to consumeImageOrNone in CSSPropertyParser

This patch renames consumeImage to consumeImageOrNone to more
accurately represent what the function does. It consumes an ident in
the case CSSValueNone is specified and consumes an image otherwise.

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

[commit-bot: I haz the power, 2016-03-30 00:58:04]

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