CC Animation: Use bitset instead of unordered_set in LayerAnimationController

CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A performance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[loyso (OOO), 2016-03-17 06:04:38]

Description was changed from

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A perfomance optimization. This avoids any heap allocations.

BUG=595584
==========

to

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A perfomance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[loyso (OOO), 2016-03-17 06:06:18]

Description was changed from

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A perfomance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A perfomance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[loyso (OOO), 2016-03-17 06:06:18]

loyso@chromium.org changed reviewers:
+ ajuma@chromium.org, vollick@chromium.org

[loyso (OOO), 2016-03-17 06:08:54]

Description was changed from

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A perfomance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A performance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[loyso (OOO), 2016-03-17 06:25:14]

Alternatively, we could use operator[]. It's more readable (if one thinks in
key-value terms)

[ajuma, 2016-03-17 13:25:13]

On 2016/03/17 06:25:14, loyso wrote:
> Alternatively, we could use operator[]. It's more readable (if one thinks in
> key-value terms)

Agreed, using operator[] instead of set/test would make this more readable.

[loyso (OOO), 2016-03-17 22:55:17]

On 2016/03/17 13:25:13, ajuma wrote:
> On 2016/03/17 06:25:14, loyso wrote:
> > Alternatively, we could use operator[]. It's more readable (if one thinks in
> > key-value terms)
> 
> Agreed, using operator[] instead of set/test would make this more readable.
Done. PTAL.

[ajuma, 2016-03-17 23:06:45]

Thanks, lgtm.

[loyso (OOO), 2016-03-17 23:39:21]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2016-03-17 23:39:44]

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

[commit-bot: I haz the power, 2016-03-18 00:36:17]

Description was changed from

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A performance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A performance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-03-18 00:36:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-18 00:37:38]

Description was changed from

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A performance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Use bitset instead of unordered_set in LayerAnimationController

A performance optimization. This avoids any heap allocations.

BUG=595584
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-03-18 00:37:39]

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