Bug 105708

Summary: [BlackBerry] Set load state for popups manually in absence of a loader
Product: WebKit Reporter: Chris.Guan <logingx>
Component: WebKit BlackBerryAssignee: Jakob Petsovits <jpetsovits>
Status: RESOLVED FIXED    
Severity: Normal CC: jpetsovits, manyoso, mifenton, rwlbuis, staikos, tonikitoo, webkit.review.bot, yong.li.webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Chris.Guan 2012-12-23 22:32:48 PST 
as title
Comment 1 Chris.Guan 2012-12-23 22:42:46 PST 
Created attachment 180644 [details]
Patch
Comment 2 Jakob Petsovits 2012-12-23 22:48:46 PST 
Comment on attachment 180644 [details]
Patch

I understand that you came up with a similar patch, but if you take mine then please don't label the commit message under your name. Instead, add yourself underneath the reviewer line with "Internally reviewed by Chris Guan."
Comment 3 Jakob Petsovits 2012-12-24 07:11:44 PST 
Created attachment 180669 [details]
Patch

Merry Christmas!
Comment 4 Charles Wei 2012-12-25 00:28:34 PST 
Comment on attachment 180669 [details]
Patch

Looks good. Need a reviewer to approve it though.
Comment 5 WebKit Review Bot 2012-12-26 22:44:59 PST 
Comment on attachment 180669 [details]
Patch

Clearing flags on attachment: 180669

Committed r138490: <http://trac.webkit.org/changeset/138490>
Comment 6 WebKit Review Bot 2012-12-26 22:45:02 PST 
All reviewed patches have been landed.  Closing bug.