Desktop chrome tab capture-chooseDesktopMedia()

content/public/browser/web_contents_media_capture_id.h

Index: content/public/browser/web_contents_media_capture_id.h
diff --git a/content/public/browser/web_contents_media_capture_id.h b/content/public/browser/web_contents_media_capture_id.h
new file mode 100644
index 0000000000000000000000000000000000000000..81b2354528939e1edb89fbe9f22727c8eb60a33d
--- /dev/null
+++ b/content/public/browser/web_contents_media_capture_id.h
@@ -0,0 +1,59 @@
+// Copyright (c) 2015 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 CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_MEDIA_CAPTURE_ID_H_
+#define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_MEDIA_CAPTURE_ID_H_
+
+#include <string>
+#include <tuple>

[miu, 2015/12/12 00:00:33]

nit: Put the <tuple> include in the .cc file since it is not used in this header
file.

[GeorgeZ, 2015/12/14 18:33:07]

Done.

+
+#include "content/common/content_export.h"
+
+namespace content {
+const char kTabPrefix[] = "web-contents-media-stream";
+
+struct CONTENT_EXPORT WebContentsMediaCaptureId {
+ public:
+  // Represents an "unset" value for either |render_process_id| or
+  // |main_render_frame_id|.
+  static const int kInvalidId = -1;
+
+  WebContentsMediaCaptureId() = default;
+  WebContentsMediaCaptureId(int render_process_id, int main_render_frame_id)
+      : render_process_id(render_process_id),
+        main_render_frame_id(main_render_frame_id) {}
+
+  bool operator<(const WebContentsMediaCaptureId& other) const;
+  bool operator==(const WebContentsMediaCaptureId& other) const;
+
+  // Return true if render_process_id or main_render_frame_id is invalid.
+  bool is_null() const;
+
+  std::string ToString() const;
+
+  // Tab video and audio capture need render process id and render frame id.
+  int render_process_id = kInvalidId;
+  int main_render_frame_id = kInvalidId;
+

[miu, 2015/12/12 00:00:33]

Please add a field for the auto-throttling option and change all the
methods/functions to account for it.  Probably a bool would suffice.

[GeorgeZ, 2015/12/14 18:33:07]

Done.

+  // Create WebContentsMediaCaptureId based on a string.
+  static WebContentsMediaCaptureId Parse(const std::string& str);
+
+  // Check whether the device id indicates that this is a web contents stream.
+  static bool IsWebContentsDeviceId(const std::string& device_id);
+
+  // Function to extract the target render frame id's from a media stream
+  // request's device id.
+  static bool ExtractTabCaptureTarget(const std::string& device_id,
+                                      int* render_process_id,
+                                      int* main_render_frame_id);
+
+  // Parses the media stream request |device_id| and returns true if both 1) the
+  // format is valid, and 2) the throttling option is set to auto.
+  static bool IsAutoThrottlingOptionSet(const std::string& device_id);
+
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_MEDIA_CAPTURE_ID_H_