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

Side by Side Diff: chrome/browser/media/media_access_handler.h

Issue 1095393004: Refactor: Make MediaCaptureDevicesDispatcher have pluggable handlers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 // Copyright (c) 2015 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 CHROME_BROWSER_MEDIA_MEDIA_ACCESS_HANDLER_H_
6 #define CHROME_BROWSER_MEDIA_MEDIA_ACCESS_HANDLER_H_
7
8 #include <deque>
9 #include <map>
10
11 #include "content/public/browser/web_contents.h"
Sergey Ulanov 2015/04/24 22:43:51 WebContents can be forward-declared
changbin 2015/04/29 05:24:34 Done.
12 #include "content/public/common/media_stream_request.h"
13
14 namespace extensions {
15 class Extension;
16 }
17
18 class MediaAccessHandler {
Sergey Ulanov 2015/04/24 22:43:51 Add comments to explain what this interface is use
changbin 2015/04/29 05:24:34 Done.
19 public:
20 MediaAccessHandler();
21 virtual ~MediaAccessHandler();
22
23 virtual void HandleRequest(content::WebContents* web_contents,
24 const content::MediaStreamRequest& request,
25 const content::MediaResponseCallback& callback,
26 const extensions::Extension* extension) = 0;
27 };
28
29 struct PendingAccessRequest {
Sergey Ulanov 2015/04/24 22:43:51 This doesn't need to be in this header - they are
changbin 2015/04/29 05:24:34 Thanks, done. PTAL
30 PendingAccessRequest(const content::MediaStreamRequest& request,
31 const content::MediaResponseCallback& callback);
32 ~PendingAccessRequest();
33
34 // TODO(gbillock): make the MediaStreamDevicesController owned by
35 // this object when we're using bubbles.
36 content::MediaStreamRequest request;
37 content::MediaResponseCallback callback;
38 };
39
40 typedef std::deque<PendingAccessRequest> RequestsQueue;
41 typedef std::map<content::WebContents*, RequestsQueue> RequestsQueues;
42
43 #endif // CHROME_BROWSER_MEDIA_MEDIA_ACCESS_HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698