[Webkit-unassigned] [Bug 29185] [Qt] QWebGraphicsItem should check for null QWebPage

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=29185


Antonio Gomes (tonikitoo) <tonikitoo at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever Confirmed|0                           |1
                 CC|                            |tonikitoo at gmail.com
             Status|UNCONFIRMED                 |NEW
           Keywords|                            |Qt




--- Comment #2 from Antonio Gomes (tonikitoo) <tonikitoo at gmail.com>  2009-09-11 09:08:34 PDT ---
jakub, looks good overall ... just one question: in comment 
https://bugs.webkit.org/show_bug.cgi?id=28862#c14 simon suggested to not use
page() (maybe to avoid a unnecessary method call overhead), but d->page ...
maybe we should respect that ?

ps: although i agree w/ your idea since page() willalways return a non-null
pointer

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.