First programmatic smooth scroll should run on the compositor if it can

third_party/WebKit/Source/core/testing/Internals.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple 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.
@@ skipping 118 matching lines @@
 #include "platform/Cursor.h"
 #include "platform/Language.h"
 #include "platform/PlatformKeyboardEvent.h"
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/TraceEvent.h"
 #include "platform/geometry/IntRect.h"
 #include "platform/geometry/LayoutRect.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/heap/Handle.h"
 #include "platform/inspector_protocol/FrontendChannel.h"
+#include "platform/scroll/ProgrammaticScrollAnimator.h"
 #include "platform/weborigin/SchemeRegistry.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebConnectionType.h"
 #include "public/platform/WebGraphicsContext3D.h"
 #include "public/platform/WebGraphicsContext3DProvider.h"
 #include "public/platform/WebLayer.h"
 #include "wtf/InstanceCounter.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/dtoa.h"
 #include "wtf/text/StringBuffer.h"
@@ skipping 2416 matching lines @@
     element->triggerAutoplayViewportCheckForTesting();
 }
 
 int Internals::getScrollAnimationState(Node* node) const
 {
     if (ScrollableArea* scrollableArea = scrollableAreaForNode(node))
         return static_cast<int>(scrollableArea->scrollAnimator().m_runState);
     return -1;
 }
 
+int Internals::getProgrammaticScrollAnimationState(Node* node) const
+{
+    if (ScrollableArea* scrollableArea = scrollableAreaForNode(node))
+        return static_cast<int>(scrollableArea->programmaticScrollAnimator().m_runState);

[jbroman, 2016/03/14 17:35:32]

This seems super-internal, even for Internals. It's already here, so I guess I
won't demand it be changed now, but I'd at least prefer a string, so we won't
have layout tests breaking everywhere if we ever reorder the enumerators.

[ymalik, 2016/03/14 21:45:40]

Done. Will change it above in a follow up CL to not clutter this change with
other layout test fixes.

+    return -1;
+}
+
 } // namespace blink