Bug 62514

Summary: [Qt] Qt Counterpart for http://trac.webkit.org/changeset/42078
Product: WebKit Reporter: Robert Hogan <robert>
Component: New BugsAssignee: Robert Hogan <robert>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, kling, ossy, rniwa, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 62926    
Bug Blocks: 62929    
Attachments:
Description Flags
Patch
none
Patch none

Description Robert Hogan 2011-06-12 10:30:48 PDT
[Qt] Qt Counterpart for http://trac.webkit.org/changeset/42078
Comment 1 Robert Hogan 2011-06-12 10:38:23 PDT
Created attachment 96882 [details]
Patch
Comment 2 Robert Hogan 2011-06-17 12:10:44 PDT
Committed r89154: <http://trac.webkit.org/changeset/89154>
Comment 3 Ryosuke Niwa 2011-06-17 16:39:15 PDT
It seems like this patch caused Qt Windows build failure:
http://build.webkit.org/builders/Qt%20Windows%2032-bit%20Debug/builds/24074/steps/compile-webkit/logs/stdio
Comment 4 Robert Hogan 2011-06-17 18:34:32 PDT
Whoops.i'll take a look in the morning.
Comment 5 Csaba Osztrogonác 2011-06-18 01:43:22 PDT
Robert, you should have watch the bots after landing instead of
waiting if somebody kick in your ass. :) Please respect 
contributing rules and try to keep the tree green:
http://www.webkit.org/coding/contributing.html

"Your change must at least compile on all platforms."

I had to roll it out to make the bots green again: http://trac.webkit.org/changeset/89191
Comment 6 Robert Hogan 2011-06-18 03:33:57 PDT
Created attachment 97695 [details]
Patch
Comment 7 Alexey Proskuryakov 2011-06-18 09:34:52 PDT
Comment on attachment 97695 [details]
Patch

ok
Comment 8 WebKit Review Bot 2011-06-19 03:53:16 PDT
Comment on attachment 97695 [details]
Patch

Clearing flags on attachment: 97695

Committed r89213: <http://trac.webkit.org/changeset/89213>
Comment 9 WebKit Review Bot 2011-06-19 03:53:21 PDT
All reviewed patches have been landed.  Closing bug.