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

Issue 12881008: Added reply/loop assets to the video player (not uploaded by CQ before). (Closed)

Created:
7 years, 9 months ago by mtomasz
Modified:
7 years, 9 months ago
Reviewers:
CC:
chromium-reviews, feature-media-reviews_chromium.org, rginda+watch_chromium.org, arv+watch_chromium.org
Visibility:
Public.

Description

Added reply/loop assets to the video player (not uploaded by CQ before). Previous try via CQ failed: https://codereview.chromium.org/12676002/ TBR=yoshiki@chromium.org TEST=Launch a video in Files.app, wait until end or scroll to the end. BUG=177591 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=188304

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+-6 lines, --6 lines) Patch
M chrome/browser/resources/file_manager/images/media/2x/media_loop.png View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/browser/resources/file_manager/images/media/2x/media_loop_down.png View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/browser/resources/file_manager/images/media/2x/media_loop_hover.png View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/browser/resources/file_manager/images/media/media_loop.png View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/browser/resources/file_manager/images/media/media_loop_down.png View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/browser/resources/file_manager/images/media/media_loop_hover.png View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
mtomasz
7 years, 9 months ago (2013-03-15 08:36:13 UTC) #1
Message was sent while issue was closed.
Committed patchset #1 manually as r188304 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698