[BlackBerry] media volume slider in wrong position (master_38 regression)
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jun 2012 14:17:29 +0000 (14:17 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jun 2012 14:17:29 +0000 (14:17 +0000)
https://bugs.webkit.org/show_bug.cgi?id=89165

Patch by Max Feil <mfeil@rim.com> on 2012-06-15
Reviewed by Antonio Gomes.

Due to upstream simplification of the volume slider (webkit.org
bug 82150) it now appears in the wrong location below the
media controls. The programmatic controls sizing we do in
RenderThemeBlackBerry.cpp relies on absolute positioning of the
volume slider container, so the best way to fix this is to set
the bottom offset.

No new tests since this is a regression fix. BlackBerry media
controls are manually tested.

* platform/blackberry/RenderThemeBlackBerry.cpp:
(WebCore::RenderThemeBlackBerry::adjustMediaControlStyle):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@120453 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/blackberry/RenderThemeBlackBerry.cpp

index 9be5c6718a3be6c1515b87b6e2aebe89edea9aa5..1bc6355218897e3bea2c7232bc60dee289873732 100644 (file)
@@ -1,3 +1,23 @@
+2012-06-15  Max Feil  <mfeil@rim.com>
+
+        [BlackBerry] media volume slider in wrong position (master_38 regression)
+        https://bugs.webkit.org/show_bug.cgi?id=89165
+
+        Reviewed by Antonio Gomes.
+
+        Due to upstream simplification of the volume slider (webkit.org
+        bug 82150) it now appears in the wrong location below the
+        media controls. The programmatic controls sizing we do in
+        RenderThemeBlackBerry.cpp relies on absolute positioning of the
+        volume slider container, so the best way to fix this is to set
+        the bottom offset.
+
+        No new tests since this is a regression fix. BlackBerry media
+        controls are manually tested.
+
+        * platform/blackberry/RenderThemeBlackBerry.cpp:
+        (WebCore::RenderThemeBlackBerry::adjustMediaControlStyle):
+
 2012-06-15  'Pavel Feldman'  <pfeldman@chromium.org>
 
         Not reviewed: remove garbage line from inspector view in docked-to-right mode.
index abfa5f09da5ef2035209fa1544edf58acdc4b27e..7e258816fa67bc1c3549a1198893b74a76a10b06 100644 (file)
@@ -776,6 +776,7 @@ void RenderThemeBlackBerry::adjustMediaControlStyle(StyleResolver*, RenderStyle*
     case MediaVolumeSliderContainerPart:
         style->setWidth(controlsHeight);
         style->setHeight(volumeHeight);
+        style->setBottom(controlsHeight);
         break;
     default:
         break;