OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2017 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef MEDIA_BASE_MOJO_ANDROID_OVERLAY_FACTORY_H_ | |
6 #define MEDIA_BASE_MOJO_ANDROID_OVERLAY_FACTORY_H_ | |
7 | |
8 #include "base/macros.h" | |
9 #include "media/base/android/android_overlay_factory.h" | |
10 #include "media/mojo/interfaces/android_overlay.mojom.h" | |
11 | |
12 namespace service_manager { | |
13 namespace mojom { | |
14 class InterfaceProvider; | |
15 } | |
16 } | |
17 | |
18 namespace media { | |
19 | |
20 // AndroidOverlayFactory implementation that memorizes the render frame with | |
21 // which the overlays are associated. | |
22 class MojoAndroidOverlayFactory : public AndroidOverlayFactory { | |
23 public: | |
24 MojoAndroidOverlayFactory( | |
25 service_manager::mojom::InterfaceProvider* interface_provider, | |
26 int render_frame_id, | |
27 int renderer_pid); | |
28 ~MojoAndroidOverlayFactory() override; | |
29 | |
30 std::unique_ptr<AndroidOverlay> CreateOverlay( | |
31 const AndroidOverlay::Config& config) override; | |
32 | |
33 private: | |
34 service_manager::mojom::InterfaceProvider* interface_provider_; | |
dcheng
2017/02/28 05:22:03
Nit: I personally prefer to mark these const to ma
liberato (no reviews please)
2017/03/06 07:51:04
Done.
| |
35 int render_frame_id_; | |
dcheng
2017/02/28 05:22:03
This feels like a layering violation. //media can'
liberato (no reviews please)
2017/03/06 07:51:05
yeah, it is. i'd like to remove them both; they'r
| |
36 int renderer_pid_; | |
37 | |
38 DISALLOW_COPY_AND_ASSIGN(MojoAndroidOverlayFactory); | |
39 }; | |
40 | |
41 } // namespace media | |
42 | |
43 #endif // MEDIA_BASE_MOJO_ANDROID_OVERLAY_H_ | |
OLD | NEW |