Removed incorrect StyleInvalidator FIXME.

Removed incorrect StyleInvalidator FIXME.

Cloning the RenderStyle, like the FIXME said, is not enough. It would be
true if we only scheduled invalidation sets for classes/ids/... not in the
rightmost compound selector, but we schedule invalidation sets for classes
on the element it needs to invalidate too. Setting class=a on an element
with the style rules below requires us to do a LocalStyleChange on both
'.a' and '.b' in order to apply the first rule as well. For setting
class=x, that wouldn't be necesarry.

.a {}
.a .b {}
.x .y {}

It's possible to optimize that case by adding a flag to the invalidation
set to indicate that the feature the invalidation set is for is present in
the rightmost compound of some selector.

R=chrishtr@chromium.org,esprehn@chromium.org

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

[esprehn, 2014-10-24 21:54:58]

lgtm, lets not optimize just yet.

[rune, 2014-10-24 22:40:39]

On 2014/10/24 at 21:54:58, esprehn wrote:
> lgtm, lets not optimize just yet.

agree

[rune, 2014-10-24 22:41:05]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-10-24 22:41:57]

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

[commit-bot: I haz the power, 2014-10-25 00:32:58]

Committed patchset #1 (id:1) as 184394