Devtools Animations: Basic animation inspection & control in Styles pane

Source/core/inspector/InspectorCSSAgent.cpp

 /*
  * Copyright (C) 2010, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *     notice, this list of conditions and the following disclaimer in the
@@ skipping 12 matching lines @@
  */
 
 #include "config.h"
 #include "core/inspector/InspectorCSSAgent.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ExceptionStatePlaceholder.h"
 #include "core/CSSPropertyNames.h"
 #include "core/InspectorTypeBuilder.h"
 #include "core/StylePropertyShorthand.h"
+#include "core/animation/Animation.h"
+#include "core/animation/AnimationEffect.h"
+#include "core/animation/AnimationNode.h"
+#include "core/animation/AnimationPlayer.h"
+#include "core/animation/ElementAnimation.h"
+#include "core/animation/KeyframeEffectModel.h"
 #include "core/css/CSSComputedStyleDeclaration.h"
 #include "core/css/CSSDefaultStyleSheets.h"
 #include "core/css/CSSImportRule.h"
 #include "core/css/CSSMediaRule.h"
 #include "core/css/CSSRule.h"
 #include "core/css/CSSRuleList.h"
 #include "core/css/CSSStyleRule.h"
 #include "core/css/CSSStyleSheet.h"
 #include "core/css/MediaList.h"
 #include "core/css/MediaQuery.h"
@@ skipping 16 matching lines @@
 #include "core/inspector/InspectorResourceContentLoader.h"
 #include "core/inspector/InspectorState.h"
 #include "core/inspector/InstrumentingAgents.h"
 #include "core/loader/DocumentLoader.h"
 #include "core/page/Page.h"
 #include "core/rendering/InlineTextBox.h"
 #include "core/rendering/RenderObject.h"
 #include "core/rendering/RenderObjectInlines.h"
 #include "core/rendering/RenderText.h"
 #include "core/rendering/RenderTextFragment.h"
+#include "platform/Decimal.h"
 #include "platform/fonts/Font.h"
 #include "platform/fonts/GlyphBuffer.h"
 #include "platform/fonts/SimpleShaper.h"
 #include "platform/text/TextRun.h"
 #include "wtf/CurrentTime.h"
 #include "wtf/text/CString.h"
 #include "wtf/text/StringConcatenate.h"
 
 namespace CSSAgentState {
 static const char cssAgentEnabled[] = "cssAgentEnabled";
@@ skipping 746 matching lines @@
     if (!inspectorStyleSheet) {
         *errorString = "Style sheet with id " + styleSheetId + " not found";
         return;
     }
 
     TrackExceptionState exceptionState;
     m_domAgent->history()->perform(adoptRefWillBeNoop(new SetStyleSheetTextAction(inspectorStyleSheet, text)), exceptionState);
     *errorString = InspectorDOMAgent::toErrorString(exceptionState);
 }
 
+void InspectorCSSAgent::getKeyframeStylesForNode(ErrorString* errorString, int sequenceNumber, RefPtr<TypeBuilder::Array<TypeBuilder::CSS::CSSRule> >& keyframeStyles)
+{
+    keyframeStyles = TypeBuilder::Array<TypeBuilder::CSS::CSSRule>::create();
+    AnimationPlayer& player = *(m_domAgent->m_idToAnimationPlayer.get(sequenceNumber));

[caseq, 2014/10/02 09:48:01]

You can't expect client to provide a valid id, please check and return protocol
error. 

[samli, 2014/10/03 06:05:04]

Done.

+    ASSERT(player.source());
+    // FIXME: Needs to consider AnimationGroup once added.
+    ASSERT(player.source()->isAnimation());
+    const Animation& animation = *toAnimation(player.source());
+    // FIXME: Needs to consider custom effects once added.
+    ASSERT(animation.effect()->isKeyframeEffectModel());
+    const KeyframeEffectModelBase& effect = *toKeyframeEffectModelBase(animation.effect());
+    WillBeHeapVector<RefPtrWillBeMember<Keyframe> > keyframes = KeyframeEffectModelBase::normalizedKeyframes(effect.getFrames());
+
+    for (WillBeHeapVector<RefPtrWillBeMember<Keyframe> >::const_iterator i = keyframes.begin(); i != keyframes.end(); ++i) {
+        if (i->get()->isStringKeyframe()) {

[caseq, 2014/10/02 09:48:01]

nit: we prefer early bail-out, i.e:

if (!i->get()->isStringKeyframe())
    continue;

[samli, 2014/10/03 06:05:04]

Done.

+            // Convert keyframe into CSSStyleRule
+            const StringKeyframe& stringKeyframe = *toStringKeyframe(i->get());
+            RefPtr<StyleRule> styleRule = StyleRule::create();
+            styleRule->setProperties(stringKeyframe.propertySet());
+            if (!m_inspectorUserAgentStyleSheet)
+                m_inspectorUserAgentStyleSheet = CSSStyleSheet::create(CSSDefaultStyleSheets::instance().defaultStyleSheet());
+            RefPtr<CSSStyleRule> cssStyleRule = CSSStyleRule::create(styleRule.get(), m_inspectorUserAgentStyleSheet.get());
+
+            // Convert offset
+            Decimal decimal = Decimal::fromDouble(stringKeyframe.offset() * 100);
+            String offset = decimal.toString();
+            offset.append("%");

[caseq, 2014/10/02 09:48:01]

unused?

[samli, 2014/10/03 06:05:04]

It's used below in the selector text. 873
This allows the keyframe offset to be displayed.

+
+            // Cannot set selector directly since isn't a valid selector and will be discarded
+            // Construct CSSRule from scratch
+            RefPtr<TypeBuilder::Array<TypeBuilder::CSS::Selector> > selectors = TypeBuilder::Array<TypeBuilder::CSS::Selector>::create();
+            selectors->addItem(TypeBuilder::CSS::Selector::create().setValue(offset).release());
+            RefPtr<TypeBuilder::CSS::SelectorList> selectorList = TypeBuilder::CSS::SelectorList::create()
+                .setSelectors(selectors)
+                .setText("")
+                .release();
+
+            RefPtr<TypeBuilder::CSS::CSSRule> cssRule = TypeBuilder::CSS::CSSRule::create()
+                .setSelectorList(selectorList)
+                .setOrigin(TypeBuilder::CSS::StyleSheetOrigin::Inspector)
+                .setStyle(bindStyleSheet(cssStyleRule->parentStyleSheet())->buildObjectForStyle(cssStyleRule->style()))
+                .release();
+
+            keyframeStyles->addItem(cssRule);
+        }
+    }
+}
+
 static bool extractRangeComponent(ErrorString* errorString, const RefPtr<JSONObject>& range, const String& component, unsigned& result)
 {
     int parsedValue = 0;
     if (!range->getNumber(component, &parsedValue) || parsedValue < 0) {
         *errorString = "range." + component + " must be a non-negative integer";
         return false;
     }
     result = parsedValue;
     return true;
 }
@@ skipping 622 matching lines @@
     visitor->trace(m_invalidatedDocuments);
     visitor->trace(m_nodeToInspectorStyleSheet);
     visitor->trace(m_documentToViaInspectorStyleSheet);
 #endif
     visitor->trace(m_inspectorUserAgentStyleSheet);
     InspectorBaseAgent::trace(visitor);
 }
 
 } // namespace blink