Use C++11 range based iteration for invalidation sets.

Use C++11 range based iteration for invalidation sets.

Shortens down the source and improves readability.
Also removed a left-over include.

R=chrishtr@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183416

[Timothy Loh, 2014-10-08 07:14:59]

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

[Timothy Loh, 2014-10-08 07:15:00]

https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature.cpp
File Source/core/css/RuleFeature.cpp (right):

https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.cpp:308: for (const auto& it : features.classes)
Should probably update variable names here (e.g. s/it/class/) and elsewhere.

[rune, 2014-10-08 07:25:21]

On 2014/10/08 at 07:15:00, timloh wrote:
>
https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature.cpp
> File Source/core/css/RuleFeature.cpp (right):
> 
>
https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature....
> Source/core/css/RuleFeature.cpp:308: for (const auto& it : features.classes)
> Should probably update variable names here (e.g. s/it/class/) and elsewhere.

Something else than "class", but yes :-)

[rune, 2014-10-08 07:35:33]

https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature.cpp
File Source/core/css/RuleFeature.cpp (right):

https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.cpp:308: for (const auto& it : features.classes)
On 2014/10/08 at 07:15:00, Timothy Loh wrote:
> Should probably update variable names here (e.g. s/it/class/) and elsewhere.

Done.

[Timothy Loh, 2014-10-08 11:26:23]

On 2014/10/08 07:35:33, rune wrote:
>
https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature.cpp
> File Source/core/css/RuleFeature.cpp (right):
> 
>
https://codereview.chromium.org/633343002/diff/1/Source/core/css/RuleFeature....
> Source/core/css/RuleFeature.cpp:308: for (const auto& it : features.classes)
> On 2014/10/08 at 07:15:00, Timothy Loh wrote:
> > Should probably update variable names here (e.g. s/it/class/) and elsewhere.
> 
> Done.

lgtm

[Mike West, 2014-10-08 12:19:54]

mkwst@chromium.org changed reviewers:
+ mkwst@chromium.org

[Mike West, 2014-10-08 12:19:54]

Drive-by <3 C++11 lgtm. :) Thanks for cleaning this up; the result is much more
readable.

[rune, 2014-10-08 13:03:02]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-10-08 13:03:43]

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

[commit-bot: I haz the power, 2014-10-08 15:07:42]

Committed patchset #2 (id:20001) as 183416

[chrishtr, 2014-10-08 16:22:09]

lgtm