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

Issue 7472040: Merge 91337 - [chromium] Media player controls do not fade out. (Closed)

Created:
9 years, 5 months ago by sjl
Modified:
9 years, 5 months ago
Reviewers:
sjl
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/782/
Visibility:
Public.

Description

Merge 91337 - [chromium] Media player controls do not fade out. https://bugs.webkit.org/show_bug.cgi?id=64837 Media controls were not fading out when the mouse leaves the video due to underlying changes in webkit media controls. Fix was to copy missing code from MediaControlRootElement to MediaControlRootElementChromium. Reviewed by Dimitri Glazkov. * html/shadow/MediaControlRootElementChromium.cpp: (WebCore::MediaControlRootElementChromium::MediaControlRootElementChromium): (WebCore::MediaControlRootElementChromium::playbackProgressed): (WebCore::MediaControlRootElementChromium::containsRelatedTarget): (WebCore::MediaControlRootElementChromium::defaultEventHandler): * html/shadow/MediaControlRootElementChromium.h: TBR=sjl@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91477

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+833 lines, -4 lines) Patch
M Source/WebCore/ChangeLog View 1 chunk +797 lines, -4 lines 0 comments Download
M Source/WebCore/html/shadow/MediaControlRootElementChromium.h View 2 chunks +5 lines, -0 lines 0 comments Download
M Source/WebCore/html/shadow/MediaControlRootElementChromium.cpp View 4 chunks +31 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
sjl
9 years, 5 months ago (2011-07-21 16:45:50 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698