Bug 89284 - Paint played and buffered ranges differently in Chrome video controls
Summary: Paint played and buffered ranges differently in Chrome video controls
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Silvia Pfeiffer
URL:
Keywords:
Depends on:
Blocks: 84672 89696
  Show dependency treegraph
 
Reported: 2012-06-16 07:55 PDT by Silvia Pfeiffer
Modified: 2012-06-21 15:34 PDT (History)
10 users (show)

See Also:


Attachments
patch for review (8.88 KB, patch)
2012-06-16 08:12 PDT, Silvia Pfeiffer
no flags Details | Formatted Diff | Diff
Patch for cq (8.82 KB, patch)
2012-06-18 18:21 PDT, Silvia Pfeiffer
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-03 (1.97 MB, application/zip)
2012-06-18 20:56 PDT, WebKit Review Bot
no flags Details
some test updates - need to get the rest (513.43 KB, patch)
2012-06-20 08:46 PDT, Silvia Pfeiffer
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-03 (947.45 KB, application/zip)
2012-06-20 13:19 PDT, WebKit Review Bot
no flags Details
includes updated tests - also checked safari build and no new regressions (747.13 KB, patch)
2012-06-20 22:51 PDT, Silvia Pfeiffer
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Silvia Pfeiffer 2012-06-16 07:55:03 PDT
As part of the Chrome video controls visual update, the buffered range that is being played is painted in a lighter color left of the position thumb and lighter right of the position thumb.
Comment 1 Silvia Pfeiffer 2012-06-16 08:12:27 PDT
Created attachment 147967 [details]
patch for review
Comment 2 Silvia Pfeiffer 2012-06-18 18:21:15 PDT
Created attachment 148219 [details]
Patch for cq
Comment 3 WebKit Review Bot 2012-06-18 20:56:15 PDT
Comment on attachment 148219 [details]
Patch for cq

Attachment 148219 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/12977483

New failing tests:
media/media-document-audio-repaint.html
fullscreen/full-screen-stacking-context.html
media/video-no-audio.html
media/controls-strict.html
media/controls-styling.html
media/video-display-toggle.html
media/audio-repaint.html
media/audio-controls-rendering.html
media/video-zoom-controls.html
http/tests/media/video-buffered-range-contains-currentTime.html
media/video-controls-rendering.html
media/controls-without-preload.html
media/media-controls-clone.html
fast/layers/video-layer.html
media/video-playing-and-pause.html
media/controls-layout-direction.html
media/controls-after-reload.html
Comment 4 WebKit Review Bot 2012-06-18 20:56:20 PDT
Created attachment 148236 [details]
Archive of layout-test-results from ec2-cr-linux-03

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-03  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 5 Silvia Pfeiffer 2012-06-20 08:46:19 PDT
Created attachment 148577 [details]
some test updates - need to get the rest
Comment 6 WebKit Review Bot 2012-06-20 13:18:54 PDT
Comment on attachment 148577 [details]
some test updates - need to get the rest

Attachment 148577 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13010019

New failing tests:
media/media-document-audio-repaint.html
fullscreen/full-screen-stacking-context.html
media/video-no-audio.html
media/controls-strict.html
media/controls-styling.html
media/video-display-toggle.html
media/audio-repaint.html
media/audio-controls-rendering.html
media/video-zoom-controls.html
http/tests/media/video-buffered-range-contains-currentTime.html
media/video-controls-rendering.html
media/controls-without-preload.html
media/media-controls-clone.html
fast/layers/video-layer.html
media/video-playing-and-pause.html
media/controls-layout-direction.html
media/controls-after-reload.html
Comment 7 WebKit Review Bot 2012-06-20 13:19:01 PDT
Created attachment 148641 [details]
Archive of layout-test-results from ec2-cr-linux-03

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-03  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 8 Silvia Pfeiffer 2012-06-20 22:51:59 PDT
Created attachment 148731 [details]
includes updated tests - also checked safari build and no new regressions
Comment 9 WebKit Review Bot 2012-06-21 10:32:32 PDT
Comment on attachment 148731 [details]
includes updated tests - also checked safari build and no new regressions

Clearing flags on attachment: 148731

Committed r120939: <http://trac.webkit.org/changeset/120939>
Comment 10 WebKit Review Bot 2012-06-21 10:32:38 PDT
All reviewed patches have been landed.  Closing bug.