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

Side by Side Diff: public/platform/WebMediaSource.h

Issue 310733002: MSE: Start removing Blink-side support for LegacyFrameProcessor (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Explicitly fail unexpected calls to old default impl Created 6 years, 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « Source/modules/mediasource/MediaSource.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 13 matching lines...) Expand all
24 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 24 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
25 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 25 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
26 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 26 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
27 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 27 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
28 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 28 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
29 */ 29 */
30 30
31 #ifndef WebMediaSource_h 31 #ifndef WebMediaSource_h
32 #define WebMediaSource_h 32 #define WebMediaSource_h
33 33
34 #include "WebCommon.h"
34 #include "WebTimeRange.h" 35 #include "WebTimeRange.h"
35 #include "WebURL.h" 36 #include "WebURL.h"
36 37
37 namespace blink { 38 namespace blink {
38 39
39 class WebSourceBuffer; 40 class WebSourceBuffer;
40 41
41 class WebMediaSource { 42 class WebMediaSource {
42 public: 43 public:
43 enum AddStatus { 44 enum AddStatus {
44 AddStatusOk, 45 AddStatusOk,
45 AddStatusNotSupported, 46 AddStatusNotSupported,
46 AddStatusReachedIdLimit 47 AddStatusReachedIdLimit
47 }; 48 };
48 49
49 enum EndOfStreamStatus { 50 enum EndOfStreamStatus {
50 EndOfStreamStatusNoError, 51 EndOfStreamStatusNoError,
51 EndOfStreamStatusNetworkError, 52 EndOfStreamStatusNetworkError,
52 EndOfStreamStatusDecodeError, 53 EndOfStreamStatusDecodeError,
53 }; 54 };
54 55
55 enum FrameProcessorChoice { 56 enum FrameProcessorChoice {
56 UseLegacyFrameProcessor, 57 UseLegacyFrameProcessor,
57 UseNewFrameProcessor 58 UseNewFrameProcessor
58 }; 59 };
59 60
60 virtual ~WebMediaSource() { } 61 virtual ~WebMediaSource() { }
61 virtual AddStatus addSourceBuffer(const WebString& type, const WebVector<Web String>& codecs, const FrameProcessorChoice, WebSourceBuffer**) = 0; 62 // FIXME: Remove addSourceBuffer() that has FrameProcessorChoice, and remove
63 // the default implementations for addSourceBuffer() once Chromium
64 // implementation of the addSourceBuffer() with no FrameProcessorChoice has
65 // landed. See http://crbug.com/249422.
66 virtual AddStatus addSourceBuffer(const WebString& type, const WebVector<Web String>& codecs, const FrameProcessorChoice, WebSourceBuffer** webSourceBuffer)
67 {
68 // This default implementation should never be called for real.
69 BLINK_ASSERT_NOT_REACHED();
70 return AddStatusReachedIdLimit;
71 }
72 virtual AddStatus addSourceBuffer(const WebString& type, const WebVector<Web String>& codecs, WebSourceBuffer** webSourceBuffer)
73 {
74 return addSourceBuffer(type, codecs, WebMediaSource::UseNewFrameProcesso r, webSourceBuffer);
75 }
76
62 virtual double duration() = 0; 77 virtual double duration() = 0;
63 virtual void setDuration(double) = 0; 78 virtual void setDuration(double) = 0;
64 virtual void markEndOfStream(EndOfStreamStatus) = 0; 79 virtual void markEndOfStream(EndOfStreamStatus) = 0;
65 virtual void unmarkEndOfStream() = 0; 80 virtual void unmarkEndOfStream() = 0;
66 }; 81 };
67 82
68 } // namespace blink 83 } // namespace blink
69 84
70 #endif 85 #endif
OLDNEW
« no previous file with comments | « Source/modules/mediasource/MediaSource.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698