Implement LayoutTestController::numberOfActiveAnimations to get a

few Layout tests passing.

Review URL: http://codereview.chromium.org/19490

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8968 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 12 insertions, 5 deletions

diff --git a/webkit/tools/layout_tests/test_lists/tests_fixable.txt b/webkit/tools/layout_tests/test_lists/tests_fixable.txt
index 3591f88..1a6a797 100644
--- a/webkit/tools/layout_tests/test_lists/tests_fixable.txt
+++ b/webkit/tools/layout_tests/test_lists/tests_fixable.txt
@@ -1055,7 +1055,7 @@ LayoutTests/security/block-test.html = FAIL
 
 // MERGE 38500:39550: New tests
 // Missing expected results
-SKIP : LayoutTests/animations/animation-drt-api.html = PASS
+SKIP LINUX MAC : LayoutTests/animations/animation-drt-api.html = PASS
 
 // MERGE 38600:38625: New tests
 // Big 4999: we need to fix this same bug in our plugin handling Apple did in
@@ -1640,9 +1640,8 @@ SKIP : LayoutTests/fast/events/pointer-events-2.html = FAIL
 SKIP : LayoutTests/fast/events/pointer-events.html = FAIL
 
 // Merge 39660:39711 - need to implement layoutTestController.numberOfActiveAnimations()
-SKIP : LayoutTests/animations/animation-controller-drt-api.html = TIMEOUT
-SKIP : LayoutTests/transitions/zero-duration-with-non-zero-delay-end.html = TIMEOUT
-
+SKIP LINUX MAC : LayoutTests/animations/animation-controller-drt-api.html = TIMEOUT
+SKIP LINUX MAC : LayoutTests/transitions/zero-duration-with-non-zero-delay-end.html = TIMEOUT
 
 // Merge 39744:39829 - new tests
 WIN LINUX : LayoutTests/fast/canvas/canvas-incremental-repaint-2.html = FAIL
@@ -1733,7 +1732,7 @@ MAC : LayoutTests/fast/forms/002.html = FAIL
 
 // Merge 40086:40124 -- new test
 // This needs layoutTestController.numberOfActiveAnimations implemented.
-LayoutTests/transitions/repeated-firing-background-color.html = FAIL TIMEOUT
+LINUX MAC : LayoutTests/transitions/repeated-firing-background-color.html = FAIL TIMEOUT
 
 // Merge 40124:40164 -- regressions
 // This test tries to remove an iframe on the page then generate the pixel
diff --git a/webkit/tools/test_shell/layout_test_controller.cc b/webkit/tools/test_shell/layout_test_controller.cc
index 3fa9a06..45c5a38 100644
--- a/webkit/tools/test_shell/layout_test_controller.cc
+++ b/webkit/tools/test_shell/layout_test_controller.cc
@@ -101,6 +101,7 @@ LayoutTestController::LayoutTestController(TestShell* shell) {
   BindMethod("pauseAnimationAtTimeOnElementWithId", &LayoutTestController::pauseAnimationAtTimeOnElementWithId);
   BindMethod("pauseTransitionAtTimeOnElementWithId", &LayoutTestController::pauseTransitionAtTimeOnElementWithId);
   BindMethod("elementDoesAutoCompleteForElementWithId", &LayoutTestController::elementDoesAutoCompleteForElementWithId);
+  BindMethod("numberOfActiveAnimations", &LayoutTestController::numberOfActiveAnimations);
 
   // The following are stubs.
   BindMethod("dumpAsWebArchive", &LayoutTestController::dumpAsWebArchive);
@@ -617,6 +618,11 @@ void LayoutTestController::elementDoesAutoCompleteForElementWithId(
       shell_->webView(), element_id));
 }
 
+void LayoutTestController::numberOfActiveAnimations(const CppArgumentList& args,
+                                                    CppVariant* result) {
+  result->Set(webkit_glue::NumberOfActiveAnimations(shell_->webView()));
+}
+
 //
 // Unimplemented stubs
 //
diff --git a/webkit/tools/test_shell/layout_test_controller.h b/webkit/tools/test_shell/layout_test_controller.h
index 5155c3e..1be0e59 100644
--- a/webkit/tools/test_shell/layout_test_controller.h
+++ b/webkit/tools/test_shell/layout_test_controller.h
@@ -139,6 +139,8 @@ class LayoutTestController : public CppBoundClass {
                                             CppVariant* result);
   void elementDoesAutoCompleteForElementWithId(const CppArgumentList& args,
                                                CppVariant* result);
+  void numberOfActiveAnimations(const CppArgumentList& args,
+                                CppVariant* result);
 
   // The following are only stubs.  TODO(pamg): Implement any of these that
   // are needed to pass the layout tests.