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

Unified Diff: trunk/src/chrome/browser/resources/file_manager/background/js/background.js

Issue 133353007: Revert 248688 "[AudioPlayer] Enable new audio-player by default" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 10 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/chrome/browser/resources/file_manager/background/js/background.js
===================================================================
--- trunk/src/chrome/browser/resources/file_manager/background/js/background.js (revision 248694)
+++ trunk/src/chrome/browser/resources/file_manager/background/js/background.js (working copy)
@@ -691,29 +691,29 @@
var audioPlayerInitializationQueue = new AsyncUtil.Queue();
audioPlayerInitializationQueue.run(function(callback) {
- // TODO(yoshiki): Remove '--file-manager-enable-new-audio-player' flag after
- // the feature is launched.
- var newAudioPlayerEnabled = true;
+ chrome.commandLinePrivate.hasSwitch(
+ 'file-manager-enable-new-audio-player',
+ function(newAudioPlayerEnabled) {
+ var audioPlayerHTML =
+ newAudioPlayerEnabled ? 'audio_player.html' : 'mediaplayer.html';
- var audioPlayerHTML =
- newAudioPlayerEnabled ? 'audio_player.html' : 'mediaplayer.html';
+ /**
+ * Audio player window create options.
+ * @type {Object}
+ */
+ var audioPlayerCreateOptions = Object.freeze({
+ type: 'panel',
+ hidden: true,
+ minHeight: newAudioPlayerEnabled ? 116 : (35 + 58),
+ minWidth: newAudioPlayerEnabled ? 292 : 280,
+ height: newAudioPlayerEnabled ? 356 : (35 + 58),
+ width: newAudioPlayerEnabled ? 292 : 280,
+ });
- /**
- * Audio player window create options.
- * @type {Object}
- */
- var audioPlayerCreateOptions = Object.freeze({
- type: 'panel',
- hidden: true,
- minHeight: newAudioPlayerEnabled ? 116 : (35 + 58),
- minWidth: newAudioPlayerEnabled ? 292 : 280,
- height: newAudioPlayerEnabled ? 356 : (35 + 58),
- width: newAudioPlayerEnabled ? 292 : 280,
- });
-
- audioPlayer = new SingletonAppWindowWrapper(audioPlayerHTML,
- audioPlayerCreateOptions);
- callback();
+ audioPlayer = new SingletonAppWindowWrapper(audioPlayerHTML,
+ audioPlayerCreateOptions);
+ callback();
+ });
});
/**
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698