Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 10 matching lines...) Expand all Loading... | |
| 21 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT | 21 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT |
| 22 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, | 22 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, |
| 23 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT | 23 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT |
| 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 SourceBufferPrivateImpl_h | 31 #ifndef WebInbandTextTrackClientImpl_h |
| 32 #define SourceBufferPrivateImpl_h | 32 #define WebInbandTextTrackClientImpl_h |
| 33 | 33 |
| 34 #include "SourceBufferPrivate.h" | 34 #include "WebInbandTextTrackClient.h" |
| 35 #include <wtf/OwnPtr.h> | 35 |
| 36 #include <wtf/PassOwnPtr.h> | 36 namespace WebCore { |
| 37 class InbandTextTrackPrivateClient; | |
| 38 } | |
| 37 | 39 |
| 38 namespace WebKit { | 40 namespace WebKit { |
| 39 | 41 |
| 40 class WebSourceBuffer; | 42 class WebInbandTextTrackClientImpl : public WebInbandTextTrackClient { |
| 43 public: | |
| 44 explicit WebInbandTextTrackClientImpl( | |
| 45 WebCore::InbandTextTrackPrivateClient* client); | |
|
acolwell GONE FROM CHROMIUM
2013/04/17 00:13:56
nit: No need for line wrap or variable name here.
Matthew Heaney (Chromium)
2013/04/18 23:15:42
Done.
| |
| 41 | 46 |
| 42 class SourceBufferPrivateImpl : public WebCore::SourceBufferPrivate { | 47 // TODO(matthewjheaney): need dtor here? |
| 43 public: | 48 //virtual ~WebInbandTextTrackPrivateClientImpl() {} |
|
acolwell GONE FROM CHROMIUM
2013/04/17 00:13:56
Yes.
Matthew Heaney (Chromium)
2013/04/18 23:15:42
Done.
| |
| 44 explicit SourceBufferPrivateImpl(PassOwnPtr<WebSourceBuffer>); | |
| 45 virtual ~SourceBufferPrivateImpl() { } | |
| 46 | 49 |
| 47 // SourceBufferPrivate methods. | 50 virtual void addWebVTTCue(double start, |
| 48 virtual PassRefPtr<WebCore::TimeRanges> buffered(); | 51 double end, |
| 49 virtual void append(const unsigned char* data, unsigned length); | 52 const WebString& id, |
| 50 virtual void abort(); | 53 const WebString& content, |
| 51 virtual bool setTimestampOffset(double); | 54 const WebString& settings); |
| 52 virtual void removedFromMediaSource(); | |
| 53 | 55 |
| 54 private: | 56 private: |
| 55 OwnPtr<WebSourceBuffer> m_sourceBuffer; | 57 WebCore::InbandTextTrackPrivateClient* client_; |
|
acolwell GONE FROM CHROMIUM
2013/04/17 00:13:56
nit: s/client_/m_client/
Matthew Heaney (Chromium)
2013/04/18 23:15:42
Done.
| |
| 56 }; | 58 }; |
| 57 | 59 |
| 60 } // namespace WebKit | |
| 58 | 61 |
| 59 } | 62 #endif // WebInbandTextTrackClientImpl_h |
| 60 | |
| 61 #endif | |
| OLD | NEW |