Title: [109907] trunk/Source/WebKit/blackberry
Revision
109907
Author
[email protected]
Date
2012-03-06 04:59:38 -0800 (Tue, 06 Mar 2012)

Log Message

[BlackBerry] function rename issue after MediaPlayerPrivateBlackBerry.[cpp\h] upstream
https://bugs.webkit.org/show_bug.cgi?id=80377

Patch by Jonathan Dong <[email protected]> on 2012-03-06
Reviewed by Rob Buis.

As some of the function names changed in class MediaPlayerPrivate
with the upstream of MediaPlayerPrivateBlackBerry.[cpp|h], we
should update WebKit/blackberry/Api/WebPage.cpp which uses these
functions accordingly.
Renamed functions in class MediaPlayerPrivate:
windowGet() -> getWindow()
windowPositionGet() -> getWindowPosition()
mmrContextNameGet() -> mmrContextName()

* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::enterFullscreenForNode):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/Api/WebPage.cpp (109906 => 109907)


--- trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2012-03-06 12:42:54 UTC (rev 109906)
+++ trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2012-03-06 12:59:38 UTC (rev 109907)
@@ -5414,14 +5414,14 @@
     if (!mmrPlayer)
         return;
 
-    Platform::Graphics::Window* window = mmrPlayer->windowGet();
+    Platform::Graphics::Window* window = mmrPlayer->getWindow();
     if (!window)
         return;
 
     unsigned x, y, width, height;
-    mmrPlayer->windowPositionGet(x, y, width, height);
+    mmrPlayer->getWindowPosition(x, y, width, height);
 
-    const char* contextName = mmrPlayer->mmrContextNameGet();
+    const char* contextName = mmrPlayer->mmrContextName();
     if (!contextName)
         return;
 

Modified: trunk/Source/WebKit/blackberry/ChangeLog (109906 => 109907)


--- trunk/Source/WebKit/blackberry/ChangeLog	2012-03-06 12:42:54 UTC (rev 109906)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2012-03-06 12:59:38 UTC (rev 109907)
@@ -1,3 +1,22 @@
+2012-03-06  Jonathan Dong  <[email protected]>
+
+        [BlackBerry] function rename issue after MediaPlayerPrivateBlackBerry.[cpp\h] upstream
+        https://bugs.webkit.org/show_bug.cgi?id=80377
+
+        Reviewed by Rob Buis.
+
+        As some of the function names changed in class MediaPlayerPrivate
+        with the upstream of MediaPlayerPrivateBlackBerry.[cpp|h], we
+        should update WebKit/blackberry/Api/WebPage.cpp which uses these
+        functions accordingly.
+        Renamed functions in class MediaPlayerPrivate:
+        windowGet() -> getWindow()
+        windowPositionGet() -> getWindowPosition()
+        mmrContextNameGet() -> mmrContextName()
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::enterFullscreenForNode):
+
 2012-03-05  Eli Fidler  <[email protected]>
 
         The minimum font size WebSetting should actually change the LOGICAL font size in WebCore.
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to