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

Side by Side Diff: content/browser/resources/media/webrtc_internals.js

Issue 213703004: Ensure that WebRTC AEC recording checkbox is not set if cancelling file dialog. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Code review fixes + rebase. Created 6 years, 9 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 | Annotate | Revision Log
« no previous file with comments | « content/browser/resources/media/dump_creator.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 var tabView = null; 5 var tabView = null;
6 var ssrcInfoManager = null; 6 var ssrcInfoManager = null;
7 var peerConnectionUpdateTable = null; 7 var peerConnectionUpdateTable = null;
8 var statsTable = null; 8 var statsTable = null;
9 var dumpCreator = null; 9 var dumpCreator = null;
10 /** A map from peer connection id to the PeerConnectionRecord. */ 10 /** A map from peer connection id to the PeerConnectionRecord. */
(...skipping 258 matching lines...) Expand 10 before | Expand all | Expand 10 after
269 * @param {!Object} data The object containing rid {number}, the render id. 269 * @param {!Object} data The object containing rid {number}, the render id.
270 */ 270 */
271 function removeGetUserMediaForRenderer(data) { 271 function removeGetUserMediaForRenderer(data) {
272 // TODO(jiayl): remove the getUserMedia info from the tabbed UI. 272 // TODO(jiayl): remove the getUserMedia info from the tabbed UI.
273 for (var i = userMediaRequests.length - 1; i >= 0; --i) { 273 for (var i = userMediaRequests.length - 1; i >= 0; --i) {
274 if (userMediaRequests[i].rid == data.rid) 274 if (userMediaRequests[i].rid == data.rid)
275 userMediaRequests.splice(i, 1); 275 userMediaRequests.splice(i, 1);
276 } 276 }
277 } 277 }
278 278
279
280 /**
281 * Notification that the AEC recording file selection dialog was cancelled,
282 * i.e. AEC has not been enabled.
283 */
284 function aecRecordingFileSelectionCancelled() {
285 dumpCreator.disableAecRecording();
286 }
287
288
279 /** 289 /**
280 * Set 290 * Set
281 */ 291 */
282 function enableAecRecording() { 292 function enableAecRecording() {
283 dumpCreator.enableAecRecording(); 293 dumpCreator.enableAecRecording();
284 } 294 }
OLDNEW
« no previous file with comments | « content/browser/resources/media/dump_creator.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698