Bug 113872

Summary: [BlackBerry] Remove defaultViewportArguments check from call site of recomputeVirtualViewportFromViewportArguments
Product: WebKit Reporter: Arvid Nilsson <anilsson>
Component: WebKit BlackBerryAssignee: Arvid Nilsson <anilsson>
Status: RESOLVED FIXED    
Severity: Normal CC: anilsson, cgarcia, mifenton, rwlbuis, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Arvid Nilsson
Reported 2013-04-03 06:21:41 PDT
The recomputeVirtualViewportFromViewportArguments method already contains the check, as pointed out by cgarcia on bug #113829 =)
Attachments
Patch (2.18 KB, patch)
2013-04-03 06:30 PDT, Arvid Nilsson 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Arvid Nilsson
Comment 1 2013-04-03 06:30:32 PDT
Created attachment 196340 [details] Patch
Carlos Garcia Campos
Comment 2 2013-04-03 06:49:37 PDT
Comment on attachment 196340 [details] Patch Perfect!
WebKit Review Bot
Comment 3 2013-04-03 07:33:00 PDT
The commit-queue encountered the following flaky tests while processing attachment 196340 [details]: fast/js/dfg-inline-resolve.html bug 113046 (author: fpizlo@apple.com) The commit-queue is continuing to process your patch.
WebKit Review Bot
Comment 4 2013-04-03 07:34:24 PDT
Comment on attachment 196340 [details] Patch Clearing flags on attachment: 196340 Committed r147552: <http://trac.webkit.org/changeset/147552>
WebKit Review Bot
Comment 5 2013-04-03 07:34:27 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.