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

Unified Diff: content/browser/resources/media/client_renderer.js

Issue 643343004: Video Capture Capabilities available on chrome://media-internals (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits @ xhwang Created 6 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/media/media_internals_unittest.cc ('k') | content/browser/resources/media/main.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/resources/media/client_renderer.js
diff --git a/content/browser/resources/media/client_renderer.js b/content/browser/resources/media/client_renderer.js
index 5cdedaabb47890be131bad8070de2fcdb862a097..61281c1b8e976a02580d4f879ca350c0380265ae 100644
--- a/content/browser/resources/media/client_renderer.js
+++ b/content/browser/resources/media/client_renderer.js
@@ -122,6 +122,44 @@ var ClientRenderer = (function() {
}
},
+ redrawVideoCaptureCapabilities: function(videoCaptureCapabilities, keys) {
+ var copyButtonElement =
+ document.getElementById('video-capture-capabilities-copy-button');
+ copyButtonElement.onclick = function() {
+ window.prompt('Copy to clipboard: Ctrl+C, Enter',
+ JSON.stringify(videoCaptureCapabilities))
+ }
+
+ var videoTableBodyElement =
+ document.getElementById('video-capture-capabilities-tbody');
+ removeChildren(videoTableBodyElement);
+
+ for (var component in videoCaptureCapabilities) {
+ var tableRow = document.createElement('tr');
+ var device = videoCaptureCapabilities[ component ];
+ for (var i in keys) {
+ var value = device[keys[i]];
+ var tableCell = document.createElement('td');
+ var cellElement;
+ if ((typeof value) == (typeof [])) {
+ cellElement = document.createElement('ul');
+ for (var i in value) {
+ var format = value[i];
+ var li = document.createElement('li');
+ li.appendChild(document.createTextNode(format))
+ cellElement.appendChild(li)
+ }
+ } else {
+ cellElement = document.createTextNode(
+ ((typeof value) == 'undefined') ? 'n/a' : value);
+ }
+ tableCell.appendChild(cellElement)
+ tableRow.appendChild(tableCell);
+ }
+ videoTableBodyElement.appendChild(tableRow);
+ }
+ },
+
redrawAudioComponentList_: function(componentType, components) {
function redrawList(renderer, baseName, element) {
var fragment = document.createDocumentFragment();
« no previous file with comments | « content/browser/media/media_internals_unittest.cc ('k') | content/browser/resources/media/main.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698