Commit 51304831 authored by Fredrik Söderqvist's avatar Fredrik Söderqvist Committed by Commit Bot

Simplify group building in KeyframeEffectModelBase::EnsureKeyframeGroups

Use a single insert() rather than find()+insert().

Change-Id: Ifb7008bc02928bea44a54183dd16f479063d87a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303435
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Reviewed-by: default avatarRobert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#799699}
parent d8ab76e9
...@@ -310,17 +310,10 @@ void KeyframeEffectModelBase::EnsureKeyframeGroups() const { ...@@ -310,17 +310,10 @@ void KeyframeEffectModelBase::EnsureKeyframeGroups() const {
zero_offset_easing = &keyframe->Easing(); zero_offset_easing = &keyframe->Easing();
for (const PropertyHandle& property : keyframe->Properties()) { for (const PropertyHandle& property : keyframe->Properties()) {
KeyframeGroupMap::iterator group_iter = keyframe_groups_->find(property); Member<PropertySpecificKeyframeGroup>& group =
PropertySpecificKeyframeGroup* group; keyframe_groups_->insert(property, nullptr).stored_value->value;
if (group_iter == keyframe_groups_->end()) { if (!group)
group = group = MakeGarbageCollected<PropertySpecificKeyframeGroup>();
keyframe_groups_
->insert(property,
MakeGarbageCollected<PropertySpecificKeyframeGroup>())
.stored_value->value.Get();
} else {
group = group_iter->value.Get();
}
Keyframe::PropertySpecificKeyframe* property_specific_keyframe = Keyframe::PropertySpecificKeyframe* property_specific_keyframe =
keyframe->CreatePropertySpecificKeyframe(property, composite_, keyframe->CreatePropertySpecificKeyframe(property, composite_,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment