Steps to Reproduce: *Open Browse *Go to http://www.cloudtweaks.com/ *Observe that the information are not fitted to the screen - able to scroll left and right, instead of just up and down Expected: *All web pages should fit in to the screen
Created attachment 147828 [details] Patch
Comment on attachment 147828 [details] Patch Will update the internal reviewer.
Created attachment 147829 [details] committed, r=rwlbuis
Comment on attachment 147829 [details] committed, r=rwlbuis View in context: https://bugs.webkit.org/attachment.cgi?id=147829&action=review LGTM. > Source/WebKit/blackberry/Api/WebPage.cpp:1683 > + return zoomToFitScale < minimumZoomToFitScale ? minimumZoomToFitScale : zoomToFitScale; Using std:min is better here as it is clearer what you are doing.
Comment on attachment 147829 [details] committed, r=rwlbuis View in context: https://bugs.webkit.org/attachment.cgi?id=147829&action=review >> Source/WebKit/blackberry/Api/WebPage.cpp:1683 >> + return zoomToFitScale < minimumZoomToFitScale ? minimumZoomToFitScale : zoomToFitScale; > > Using std:min is better here as it is clearer what you are doing. Thanks for catching this! Should use std::max instead of std:min actually.
Comment on attachment 147829 [details] committed, r=rwlbuis Committed r120473: <http://trac.webkit.org/changeset/120473>.