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

Unified Diff: ui/file_manager/video_player/js/background.js

Issue 571453002: Correct indentation, JSDoc, etc... to comply with closure linter. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 3 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
Index: ui/file_manager/video_player/js/background.js
diff --git a/ui/file_manager/video_player/js/background.js b/ui/file_manager/video_player/js/background.js
index 6c350aed8d6d23e4d16c1b60d62cd2492747ebb9..3e97266fb50e4bca8b49939c754ecf1a4989ef3e 100644
--- a/ui/file_manager/video_player/js/background.js
+++ b/ui/file_manager/video_player/js/background.js
@@ -68,7 +68,7 @@ function onLaunched(launchData) {
/**
* Opens player window.
* @param {Array.<Object>} videos List of videos to play.
- * @param {Promise} Promise to be fulfilled on success, or rejected on error.
+ * @return {Promise} Promise to be fulfilled on success, or rejected on error.
*/
function open(videos) {
return new Promise(function(fulfill, reject) {
@@ -100,6 +100,7 @@ function open(videos) {
// If is is run in the browser test, wait for the test resources are installed
// as a component extension, and then load the test resources.
if (chrome.test) {
+ /** @type {string} */
window.testExtensionId = 'ljoplibgfehghmibaoaepfagnmbbfiga';
chrome.runtime.onMessageExternal.addListener(function(message) {
if (message.name !== 'testResourceLoaded')
« no previous file with comments | « ui/file_manager/image_loader/request.js ('k') | ui/file_manager/video_player/js/cast/cast_extension_discoverer.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698