Handle media access permission request

android_webview/native/permission/media_access_permission_request.h

+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef ANDROID_WEBVIEW_NATIVE_PERMISSION_MEDIA_ACCESS_PERMISSION_REQUEST_H
+#define ANDROID_WEBVIEW_NATIVE_PERMISSION_MEDIA_ACCESS_PERMISSION_REQUEST_H
+
+#include "android_webview/native/permission/aw_permission_request_delegate.h"
+#include "content/public/common/media_stream_request.h"
+
+namespace android_webview {
+
+// The AwPermissionRequestDelegate implementation for media access permission
+// request.
+class MediaAccessPermissionRequest : public AwPermissionRequestDelegate {

[mkosiba (inactive), 2014/04/17 10:34:06]

same here, you should be able to easily add a c++ unittest

[michaelbai, 2014/04/22 20:53:51]

Actually, it is not easy to write a unit test when you need some content
features. We need have a class subclass from RenderViewHostTestHarness to take
advantage the content testing features.

+ public:
+  MediaAccessPermissionRequest(const content::MediaStreamRequest& request,
+                               const content::MediaResponseCallback& callback);
+  virtual ~MediaAccessPermissionRequest();
+
+  // AwPermissionRequestDelegate implementation.
+  virtual const GURL& GetOrigin() OVERRIDE;
+  virtual int64 GetResources() OVERRIDE;
+  virtual void OnRequestResult(bool allowed) OVERRIDE;
+
+ private:
+  const content::MediaStreamRequest request_;

[benm (inactive), 2014/04/16 14:51:10]

nit: did you mean for these to be const?

[michaelbai, 2014/04/22 20:53:51]

Yes, I want it to be const

On 2014/04/16 14:51:10, benm wrote:

+  const content::MediaResponseCallback callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(MediaAccessPermissionRequest);
+};
+
+}  // namespace android_webview
+
+#endif  // ANDROID_WEBVIEW_NATIVE_PERMISSION_MEDIA_ACCESS_PERMISSION_REQUEST_H