Changeset 287980 in webkit

Timestamp:

Jan 13, 2022, 9:16:07 AM (5 years ago)

Author:

Peng Liu

Message:

Clean up MediaPlaybackTargetPicker::Client
​https://bugs.webkit.org/show_bug.cgi?id=235128

Reviewed by Eric Carlson.

Small cleanup to remove an unused function and change override to final.

• Modules/airplay/WebMediaSessionManager.h:
• platform/graphics/MediaPlaybackTargetPicker.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• Modules/airplay/WebMediaSessionManager.h (modified) (1 diff)
• platform/graphics/MediaPlaybackTargetPicker.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2022-01-13  Peng Liu  <peng.liu6@apple.com>
+
+        Clean up MediaPlaybackTargetPicker::Client
+        https://bugs.webkit.org/show_bug.cgi?id=235128
+
+        Reviewed by Eric Carlson.
+
+        Small cleanup to remove an unused function and change override to final.
+
+        * Modules/airplay/WebMediaSessionManager.h:
+        * platform/graphics/MediaPlaybackTargetPicker.h:
+
 2022-01-13  Ziran Sun  <zsun@igalia.com>
 

trunk/Source/WebCore/Modules/airplay/WebMediaSessionManager.h

@@ -76 +76 @@
 
     // MediaPlaybackTargetPicker::Client
-    void setPlaybackTarget(Ref<WebCore::MediaPlaybackTarget>&&) override;
-    void externalOutputDeviceAvailableDidChange(bool) override;
-    void playbackTargetPickerWasDismissed() override;
+    void setPlaybackTarget(Ref<WebCore::MediaPlaybackTarget>&&) final;
+    void externalOutputDeviceAvailableDidChange(bool) final;
+    void playbackTargetPickerWasDismissed() final;
 
     size_t find(WebMediaSessionManagerClient*, PlaybackTargetClientContextIdentifier);

trunk/Source/WebCore/platform/graphics/MediaPlaybackTargetPicker.h

@@ -48 +48 @@
         virtual void externalOutputDeviceAvailableDidChange(bool) = 0;
         virtual void playbackTargetPickerWasDismissed() = 0;
-
-        void invalidate();
     };