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

Side by Side Diff: Source/modules/encryptedmedia/MediaKeysController.h

Issue 195053004: Create Pepper-based CDM for EME WD (Blink changes) (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 9 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
OLDNEW
(Empty)
1 /*
ddorwin 2014/03/12 01:01:09 ditto
jrummell 2014/03/17 20:48:37 Done.
2 * Copyright (C) 2014 Google Inc. All rights reserved.
3 *
4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are
6 * met:
7 *
8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above
11 * copyright notice, this list of conditions and the following disclaimer
12 * in the documentation and/or other materials provided with the
13 * distribution.
14 * * Neither the name of Google Inc. nor the names of its
15 * contributors may be used to endorse or promote products derived from
16 * this software without specific prior written permission.
17 *
18 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
19 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
20 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
21 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
22 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
23 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
24 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
25 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
26 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
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.
29 */
30
31 #ifndef MediaKeysController_h
32 #define MediaKeysController_h
33
34 #include "core/page/Page.h"
35 #include "modules/encryptedmedia/MediaKeysClient.h"
ddorwin 2014/03/12 01:01:09 fwd declare
jrummell 2014/03/17 20:48:37 Done.
36 #include "wtf/PassOwnPtr.h"
37
38 namespace WebCore {
39
40 class MediaKeysController FINAL : public Supplement<Page> {
ddorwin 2014/03/12 01:01:09 I wonder whether Controller is the right name. Do
jrummell 2014/03/17 20:48:37 Most of the controllers I looked at (UserMediaCont
41 public:
42 virtual ~MediaKeysController();
43 MediaKeysClient* client() const { return m_client; }
ddorwin 2014/03/12 01:01:09 Is use of "const" common? We usually don't use con
jrummell 2014/03/17 20:48:37 I just copied what was in UserMediaController.h. C
ddorwin 2014/03/17 21:47:43 Yeah, I don't see an advantage to having the clien
jrummell 2014/03/17 22:42:13 Done. Converted to a passthru method rather than e
44
45 static PassOwnPtr<MediaKeysController> create(MediaKeysClient*);
46 static const char* supplementName();
47 static MediaKeysController* from(Page* page) { return static_cast<MediaKeysC ontroller*>(Supplement<Page>::from(page, supplementName())); }
48
49 protected:
50 explicit MediaKeysController(MediaKeysClient*);
51
52 private:
53 MediaKeysClient* m_client;
54 };
55
56 } // namespace WebCore
57
58 #endif // MediaKeysController_h
59
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698