Bug 97770

Summary: [EFL] Remove duplicated CSS between mediaControlsEfl.css and mediaControlsEflFullscreen.css
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit EFLAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarcelo, eric.carlson, feature-media-reviews, gyuyoung.kim, kenneth, lucas.de.marchi, macpherson, menard, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Chris Dumez 2012-09-27 03:36:24 PDT 
Some CSS rules are duplicated between mediaControlsEfl.css and mediaControlsEflFullscreen.css for no reason.

This is an issue because it is easy to update mediaControlsEfl.css and forget to make the same update to mediaControlsEflFullscreen.css. As a matter of fact, the timeline display in fullscreen is currently off by a few pixels because its fullscreen CSS is not in sync with what is in mediaControlsEfl.css.

We need to include in mediaControlsEflFullscreen.css only the CSS rules that are specific to fullscreen mode, that is to say, the hiding of some controls.
Comment 1 Chris Dumez 2012-09-27 03:41:02 PDT 
Created attachment 165968 [details]
Patch
Comment 2 WebKit Review Bot 2012-09-27 04:45:46 PDT 
Comment on attachment 165968 [details]
Patch

Clearing flags on attachment: 165968

Committed r129754: <http://trac.webkit.org/changeset/129754>
Comment 3 WebKit Review Bot 2012-09-27 04:45:50 PDT 
All reviewed patches have been landed.  Closing bug.