Skip to content

Commit aa79ccb

Browse files
Ahmad-S792Ahmad Saleem
authored andcommitted
Remove dead declarations from VTTCue and LoadableTextTrack headers
https://bugs.webkit.org/show_bug.cgi?id=318301 rdar://181098694 Reviewed by Simon Fraser. Remove three declarations that have no definition and no callers. * html/track/VTTCue.h: - Remove unused m_cue member; VTTCueBox forwards its cue to the base TextTrackCueBox and reads it back via getCue(). - Remove undefined parseSettings() declaration. * html/track/LoadableTextTrack.h: Remove undefined loadTimerFired() declaration. Canonical link: https://commits.webkit.org/316315@main
1 parent 4e721b0 commit aa79ccb

2 files changed

Lines changed: 0 additions & 7 deletions

File tree

Source/WebCore/html/track/LoadableTextTrack.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,6 @@ class LoadableTextTrack final : public TextTrack, private TextTrackLoaderClient
5858
AtomString id() const final;
5959
bool isDefault() const final;
6060

61-
void loadTimerFired();
62-
6361
#if !RELEASE_LOG_DISABLED
6462
ASCIILiteral logClassName() const override { return "LoadableTextTrack"_s; }
6563
#endif

Source/WebCore/html/track/VTTCue.h

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,9 +103,6 @@ class VTTCueBox : public TextTrackCueBox {
103103
VTTCueBox(Document&, VTTCue&);
104104

105105
RenderPtr<RenderElement> createElementRenderer(Style::ComputedStyle&&, const RenderTreePosition&) final;
106-
107-
private:
108-
WeakPtr<VTTCue> m_cue;
109106
};
110107

111108
// ----------------------------
@@ -239,8 +236,6 @@ class VTTCue
239236

240237
void createWebVTTNodeTree();
241238

242-
void parseSettings(const String&);
243-
244239
void determineTextDirection();
245240
void calculateDisplayParameters();
246241
void calculateDisplayParametersWithRegion();

0 commit comments

Comments
 (0)