Redraw media volume slider when it is moving

third_party/WebKit/Source/core/html/shadow/MediaControlsTest.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/html/shadow/MediaControls.h"
 
 #include <limits>
 #include <memory>
 #include "core/HTMLNames.h"
 #include "core/css/StylePropertySet.h"
 #include "core/dom/Document.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/StyleEngine.h"
 #include "core/frame/Settings.h"
 #include "core/html/HTMLVideoElement.h"
 #include "core/html/shadow/MediaControlElementTypes.h"
+#include "core/layout/LayoutObject.h"
 #include "core/loader/EmptyClients.h"
 #include "core/testing/DummyPageHolder.h"
 #include "platform/heap/Handle.h"
 #include "platform/testing/EmptyWebMediaPlayer.h"
 #include "platform/testing/HistogramTester.h"
 #include "platform/testing/UnitTestHelpers.h"
 #include "public/platform/WebSize.h"
 #include "public/platform/modules/remoteplayback/WebRemotePlaybackAvailability.h"
 #include "public/platform/modules/remoteplayback/WebRemotePlaybackClient.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 21 matching lines @@
   void promptCancelled() override {}
   bool remotePlaybackAvailable() const override {
     return m_availability == WebRemotePlaybackAvailability::DeviceAvailable;
   }
 
  private:
   WebRemotePlaybackAvailability m_availability =
       WebRemotePlaybackAvailability::Unknown;
 };
 
+class MockLayoutObject : public LayoutObject {
+ public:
+  MockLayoutObject() : LayoutObject(nullptr) {}
+
+  const char* name() const override { return "MockLayoutObject"; }
+  void layout() override {}
+  FloatRect localBoundingBoxRectForAccessibility() const override {
+    return FloatRect();
+  }
+
+  void setShouldDoFullPaintInvalidation(PaintInvalidationReason) {
+    m_fullPaintInvalidationCallCount++;
+  }
+
+  int fullPaintInvalidationCallCount() const {
+    return m_fullPaintInvalidationCallCount;
+  }
+
+ private:
+  int m_fullPaintInvalidationCallCount = 0;
+};
+
 class StubLocalFrameClient : public EmptyLocalFrameClient {
  public:
   static StubLocalFrameClient* create() { return new StubLocalFrameClient; }
 
   std::unique_ptr<WebMediaPlayer> createWebMediaPlayer(
       HTMLMediaElement&,
       const WebMediaPlayerSource&,
       WebMediaPlayerClient*) override {
     return WTF::wrapUnique(new MockVideoWebMediaPlayer);
   }
@@ skipping 426 matching lines @@
   double duration = 600;
   loadMediaWithDuration(duration);
 
   // Simulate seeking the underlying range to 50%. Current time display should
   // update synchronously (rather than waiting for media to finish seeking).
   timeline->setValueAsNumber(duration / 2, ASSERT_NO_EXCEPTION);
   timeline->dispatchInputEvent();
   EXPECT_EQ(duration / 2, currentTimeDisplay->currentValue());
 }
 
+TEST_F(MediaControlsTest, VolumeSliderPaintInvalidationOnInput) {
+  ensureSizing();
+
+  MediaControlVolumeSliderElement* volumeSlider =
+      static_cast<MediaControlVolumeSliderElement*>(getElementByShadowPseudoId(
+          mediaControls(), "-webkit-media-controls-volume-slider"));
+  ASSERT_NE(nullptr, volumeSlider);
+
+  MockLayoutObject layoutObject;
+  LayoutObject* prevLayoutObject = volumeSlider->layoutObject();
+  volumeSlider->setLayoutObject(&layoutObject);
+
+  volumeSlider->setVolume(0.5);

[mlamouri (slow - plz ping), 2017/03/31 11:20:16]

I'm a bit confused: setVolume() doesn't call the code path that you are
changing. Am I missing something?

Would it work if you set the value of the slider then call defaultEventHandler()
with a fake input event?

[sabbakumov, 2017/04/03 04:12:45]

Done.

+  EXPECT_EQ(1, layoutObject.fullPaintInvalidationCallCount());
+
+  volumeSlider->setVolume(0.3);
+  EXPECT_EQ(2, layoutObject.fullPaintInvalidationCallCount());
+
+  volumeSlider->setVolume(0.1);
+  EXPECT_EQ(3, layoutObject.fullPaintInvalidationCallCount());
+
+  volumeSlider->setLayoutObject(prevLayoutObject);
+}
+
 }  // namespace blink