Bug 77752

Summary: Extra TestWebKitAPI.gyp/TestWebKitAPI.target.chromium.mk in repository
Product: WebKit Reporter: Xianzhu Wang <wangxianzhu>
Component: Tools / TestsAssignee: Dirk Pranke <dpranke>
Status: RESOLVED FIXED    
Severity: Normal CC: dpranke, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch
none
Sorry for uploading patch to wrong bug. Please ignore this.
none
Patch none

Description Xianzhu Wang 2012-02-03 10:41:10 PST
<EOM>
Comment 1 Xianzhu Wang 2012-02-03 10:43:11 PST
Created attachment 125362 [details]
patch
Comment 2 Xianzhu Wang 2012-02-03 10:49:19 PST
Created attachment 125364 [details]
Sorry for uploading patch to wrong bug. Please ignore this.

Thanks for suggestions. Not using ASSERT because I don't want the callers to know how viewport update and the main frame are related and to check it.
Comment 3 Dirk Pranke 2012-02-03 12:48:07 PST
Created attachment 125386 [details]
Patch
Comment 4 Xianzhu Wang 2012-02-03 14:09:58 PST
(In reply to comment #3)
> Created an attachment (id=125386) [details]
> Patch

Look good to me, but need r+ from some reviewer.
Comment 5 WebKit Review Bot 2012-02-03 15:53:41 PST
Comment on attachment 125386 [details]
Patch

Clearing flags on attachment: 125386

Committed r106702: <http://trac.webkit.org/changeset/106702>
Comment 6 WebKit Review Bot 2012-02-03 15:53:45 PST
All reviewed patches have been landed.  Closing bug.