Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(899)

Unified Diff: Source/core/loader/TextTrackLoader.h

Issue 25798003: Enable WebVTT regions for runtime testing, updated tests and minor fixes (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/html/track/WebVTTParser.cpp ('k') | Source/core/loader/TextTrackLoader.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/loader/TextTrackLoader.h
diff --git a/Source/core/loader/TextTrackLoader.h b/Source/core/loader/TextTrackLoader.h
index f76a67b899ad01be93d8118ecacd9ef216d710ed..fd8f5e8e2e637f9e3c62c4ee76ba5c4578bfd117 100644
--- a/Source/core/loader/TextTrackLoader.h
+++ b/Source/core/loader/TextTrackLoader.h
@@ -47,9 +47,7 @@ public:
virtual void newCuesAvailable(TextTrackLoader*) = 0;
virtual void cueLoadingStarted(TextTrackLoader*) = 0;
virtual void cueLoadingCompleted(TextTrackLoader*, bool loadingFailed) = 0;
-#if ENABLE(WEBVTT_REGIONS)
virtual void newRegionsAvailable(TextTrackLoader*) = 0;
-#endif
};
class TextTrackLoader : public ResourceClient, private WebVTTParserClient {
@@ -69,9 +67,7 @@ public:
State loadState() { return m_state; }
void getNewCues(Vector<RefPtr<TextTrackCue> >& outputCues);
-#if ENABLE(WEBVTT_REGIONS)
void getNewRegions(Vector<RefPtr<TextTrackRegion> >& outputRegions);
-#endif
private:
// ResourceClient
@@ -80,9 +76,7 @@ private:
// WebVTTParserClient
virtual void newCuesParsed();
-#if ENABLE(WEBVTT_REGIONS)
virtual void newRegionsParsed();
-#endif
virtual void fileFailedToParse();
TextTrackLoader(TextTrackLoaderClient*, ExecutionContext*);
« no previous file with comments | « Source/core/html/track/WebVTTParser.cpp ('k') | Source/core/loader/TextTrackLoader.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698