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

Issue 6969027: Trigger GPU acceleration in the picture slide show and the media player. (Closed)

Created:
9 years, 7 months ago by marcheu
Modified:
9 years, 7 months ago
Reviewers:
achuithb, zel
CC:
chromium-reviews, rginda+watch_chromium.org, arv (Not doing code reviews), achuith+watch_chromium.org
Visibility:
Public.

Description

Trigger GPU acceleration in the picture slide show and the media player. This change adds a noop (-webkit-transform: translateZ(0)) property to the file manager's picture slideshow and to the media player. This triggers GPU acceleration and avoids tearing in Chrome OS. BUG=chromium-os:14881 TEST=Run the picture slide show, go from one picture to the other and check that there is no tearing.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M chrome/browser/resources/file_manager/slideshow.html View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/resources/mediaplayer.html View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
marcheu
9 years, 7 months ago (2011-05-12 02:05:08 UTC) #1
zel
LGTM
9 years, 7 months ago (2011-05-12 16:40:37 UTC) #2
marcheu
On 2011/05/12 16:40:37, zel wrote: > LGTM Zel can you please push this? I don't ...
9 years, 7 months ago (2011-05-12 19:23:07 UTC) #3
achuithb
I can do it. On 2011/05/12 19:23:07, marcheu wrote: > On 2011/05/12 16:40:37, zel wrote: ...
9 years, 7 months ago (2011-05-12 19:43:39 UTC) #4
achuithb
http://src.chromium.org/viewvc/chrome?view=rev&revision=85170 On 2011/05/12 19:43:39, achuith.bhandarkar wrote: > I can do it. > > On 2011/05/12 ...
9 years, 7 months ago (2011-05-12 20:01:58 UTC) #5
marcheu
9 years, 7 months ago (2011-05-12 20:30:29 UTC) #6
thanks Achuith!

Powered by Google App Engine
This is Rietveld 408576698