Title: [136946] trunk/Source/WebCore
Revision
136946
Author
[email protected]
Date
2012-12-07 04:57:51 -0800 (Fri, 07 Dec 2012)

Log Message

[BlackBerry] make "isVideo" information available to platform media player
https://bugs.webkit.org/show_bug.cgi?id=104334

Patch by Max Feil <[email protected]> on 2012-12-07
Reviewed by Rob Buis.

The platform player needs to know whether the element is <video>
or <audio>, so it can make certain decisions before metadata
is ready. This is part of a performance optimization refactor
which does not change functionality so no new tests are required.

* platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp:
(WebCore::MediaPlayerPrivate::load):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (136945 => 136946)


--- trunk/Source/WebCore/ChangeLog	2012-12-07 12:47:24 UTC (rev 136945)
+++ trunk/Source/WebCore/ChangeLog	2012-12-07 12:57:51 UTC (rev 136946)
@@ -1,3 +1,18 @@
+2012-12-07  Max Feil  <[email protected]>
+
+        [BlackBerry] make "isVideo" information available to platform media player
+        https://bugs.webkit.org/show_bug.cgi?id=104334
+
+        Reviewed by Rob Buis.
+
+        The platform player needs to know whether the element is <video>
+        or <audio>, so it can make certain decisions before metadata
+        is ready. This is part of a performance optimization refactor
+        which does not change functionality so no new tests are required.
+
+        * platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp:
+        (WebCore::MediaPlayerPrivate::load):
+
 2012-12-07  Kondapally Kalyan  <[email protected]>
 
         [EFL] [AC] Implement ImageExtractor::extractImage in GraphicsContext3DEfl.

Modified: trunk/Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp (136945 => 136946)


--- trunk/Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp	2012-12-07 12:47:24 UTC (rev 136945)
+++ trunk/Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp	2012-12-07 12:57:51 UTC (rev 136946)
@@ -161,12 +161,13 @@
 
     void* tabId = m_webCorePlayer->mediaPlayerClient()->mediaPlayerHostWindow()->platformPageClient();
     int playerID = m_webCorePlayer->mediaPlayerClient()->mediaPlayerHostWindow()->platformPageClient()->playerID();
+    bool isVideo = m_webCorePlayer->mediaPlayerClient()->mediaPlayerIsVideo();
 
     deleteGuardedObject(m_platformPlayer);
 #if USE(ACCELERATED_COMPOSITING)
-    m_platformPlayer = PlatformPlayer::create(this, tabId, true, modifiedUrl.utf8().data());
+    m_platformPlayer = PlatformPlayer::create(this, tabId, isVideo, true, modifiedUrl.utf8().data());
 #else
-    m_platformPlayer = PlatformPlayer::create(this, tabId, false, modifiedUrl.utf8().data());
+    m_platformPlayer = PlatformPlayer::create(this, tabId, isVideo, false, modifiedUrl.utf8().data());
 #endif
 
     WTF::String cookiePairs;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to