Enable looping of videos in Files.app.

Enable looping of videos in Files.app.

By clicking the play button or the video while holding CTRL, the video will play in a looped mode. The mode is reset after pausing.

TEST=Go to Files.app and play a video. Hit the play button, or the video area with CTRL pressed. A message should be shown and the video should get looped.
BUG=249238
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207525

[mtomasz, 2013-06-20 08:58:16]

@hashimoto: PTAL at file_browser_private_api.cc.
@hirono: PTAL at the rest.

This is urgent. Thanks!

[hashimoto, 2013-06-20 08:59:34]

file_browser_private_api.cc lgtm

[hirono, 2013-06-20 09:20:46]

lgtm with nit. Thanks!

https://codereview.chromium.org/17153007/diff/1/chrome/app/generated_resource...
File chrome/app/generated_resources.grd (right):

https://codereview.chromium.org/17153007/diff/1/chrome/app/generated_resource...
chrome/app/generated_resources.grd:12635: +        This video will keep playing
until the cows come home.
I was surprised at this message.
I wondered why cows was mentioned here.
The crbug.com/249238 page let me know the all :)

https://codereview.chromium.org/17153007/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/css/media_controls.css (right):

https://codereview.chromium.org/17153007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/media_controls.css:545: margin-left:
-250px;
Is it OK for RTL mode?

[mtomasz, 2013-06-20 09:31:59]

https://codereview.chromium.org/17153007/diff/1/chrome/app/generated_resource...
File chrome/app/generated_resources.grd (right):

https://codereview.chromium.org/17153007/diff/1/chrome/app/generated_resource...
chrome/app/generated_resources.grd:12635: +        This video will keep playing
until the cows come home.
On 2013/06/20 09:20:46, hirono wrote:
> I was surprised at this message.
> I wondered why cows was mentioned here.
> The crbug.com/249238 page let me know the all :)

I'm curious about translations :)

https://codereview.chromium.org/17153007/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/css/media_controls.css (right):

https://codereview.chromium.org/17153007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/media_controls.css:545: margin-left:
-250px;
On 2013/06/20 09:20:46, hirono wrote:
> Is it OK for RTL mode?

It is fine, just centering. I've just double checked. However, the controls in
the video player are completely broken in RTL. We have to fix all RTL stuff in
Files.app some day. There is a bug for it.

[commit-bot: I haz the power, 2013-06-20 09:35:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/17153007/1

[commit-bot: I haz the power, 2013-06-20 20:03:29]

Change committed as 207525