Title: [137932] trunk/Source/WebKit/blackberry
Revision
137932
Author
[email protected]
Date
2012-12-17 12:37:38 -0800 (Mon, 17 Dec 2012)

Log Message

[BlackBerry] Use application size rather than screen size where appropriate.
https://bugs.webkit.org/show_bug.cgi?id=105201

Patch by Andrew Lo <[email protected]> on 2012-12-17
Reviewed by Rob Buis.
Internally reviewed by Jeff Rogers.

Switch from screenSize to applicationSize where appropriate.
Internal PR266400

* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::transformedViewportSize):
(BlackBerry::WebKit::WebPagePrivate::setDefaultLayoutSize):
* WebKitSupport/SurfacePool.cpp:
(BlackBerry::WebKit::SurfacePool::initialize):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/Api/WebPage.cpp (137931 => 137932)


--- trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2012-12-17 20:30:03 UTC (rev 137931)
+++ trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2012-12-17 20:37:38 UTC (rev 137932)
@@ -1828,7 +1828,7 @@
 
 IntSize WebPagePrivate::transformedViewportSize() const
 {
-    return Platform::Graphics::Screen::primaryScreen()->size();
+    return BlackBerry::Platform::Settings::instance()->applicationSize();
 }
 
 IntRect WebPagePrivate::transformedVisibleContentsRect() const
@@ -3877,7 +3877,7 @@
 
 void WebPagePrivate::setDefaultLayoutSize(const IntSize& size)
 {
-    IntSize screenSize = Platform::Graphics::Screen::primaryScreen()->size();
+    IntSize screenSize = Platform::Settings::instance()->applicationSize();
     ASSERT(size.width() <= screenSize.width() && size.height() <= screenSize.height());
     m_defaultLayoutSize = size.expandedTo(minimumLayoutSize).shrunkTo(screenSize);
 }

Modified: trunk/Source/WebKit/blackberry/ChangeLog (137931 => 137932)


--- trunk/Source/WebKit/blackberry/ChangeLog	2012-12-17 20:30:03 UTC (rev 137931)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2012-12-17 20:37:38 UTC (rev 137932)
@@ -1,3 +1,20 @@
+2012-12-17  Andrew Lo  <[email protected]>
+
+        [BlackBerry] Use application size rather than screen size where appropriate.
+        https://bugs.webkit.org/show_bug.cgi?id=105201
+
+        Reviewed by Rob Buis.
+        Internally reviewed by Jeff Rogers.
+
+        Switch from screenSize to applicationSize where appropriate.
+        Internal PR266400
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::transformedViewportSize):
+        (BlackBerry::WebKit::WebPagePrivate::setDefaultLayoutSize):
+        * WebKitSupport/SurfacePool.cpp:
+        (BlackBerry::WebKit::SurfacePool::initialize):
+
 2012-12-17  Mike Fenton  <[email protected]>
 
         [BlackBerry] Improve DOMSupport visibleSelectionForClosestActualWordStart for content editable.

Modified: trunk/Source/WebKit/blackberry/WebKitSupport/SurfacePool.cpp (137931 => 137932)


--- trunk/Source/WebKit/blackberry/WebKitSupport/SurfacePool.cpp	2012-12-17 20:30:03 UTC (rev 137931)
+++ trunk/Source/WebKit/blackberry/WebKitSupport/SurfacePool.cpp	2012-12-17 20:37:38 UTC (rev 137932)
@@ -79,7 +79,7 @@
     const unsigned maxNumberOfTiles = Platform::Settings::instance()->maximumNumberOfBackingStoreTilesAcrossProcesses();
 
     if (m_numberOfFrontBuffers) { // Only allocate if we actually use a backingstore.
-        Platform::IntSize screenSize = Platform::Graphics::Screen::primaryScreen()->size();
+        Platform::IntSize screenSize = BlackBerry::Platform::Settings::instance()->applicationSize();
         unsigned byteLimit = maxNumberOfTiles * tileSize.width() * tileSize.height() * 4;
         byteLimit += screenSize.width() * screenSize.height() * 4; // visible tile buffer - FIXME, fragile for further maintenance as its size doesn't sync up with the rest
         bool success = Platform::Graphics::createPixmapGroup(SHARED_PIXMAP_GROUP, byteLimit);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to