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

Issue 385113007: Video Player: Automatically load next video (Closed)

Created:
6 years, 5 months ago by etrunko
Modified:
6 years, 5 months ago
Reviewers:
yoshiki
CC:
chromium-reviews, rginda+watch_chromium.org, mtomasz+watch_chromium.org, yoshiki+watch_chromium.org, nathan ciobanu
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Video Player: Automatically load next video Whenever a video playback finishes, the users are required to manually skip to the next video on the list. This patch enables the next video to be loaded automatically, but only if the current one is not playing in loop mode. BUG=393318 R=yoshiki@chromium.org TEST=manually tested Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>; Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282966

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
M ui/file_manager/video_player/js/video_player.js View 1 chunk +9 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
etrunko
6 years, 5 months ago (2014-07-11 20:49:21 UTC) #1
yoshiki
lgtm. Thanks for doing this!
6 years, 5 months ago (2014-07-14 02:17:59 UTC) #2
etrunko
The CQ bit was checked by eduardo.lima@intel.com
6 years, 5 months ago (2014-07-14 13:14:47 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/eduardo.lima@intel.com/385113007/1
6 years, 5 months ago (2014-07-14 13:15:26 UTC) #4
commit-bot: I haz the power
6 years, 5 months ago (2014-07-14 17:14:29 UTC) #5
Message was sent while issue was closed.
Change committed as 282966

Powered by Google App Engine
This is Rietveld 408576698