Bug 19374

Summary: Expose boundingBox value in QWebHitTestResult
Product: WebKit Reporter: Siraj razick <siraj>
Component: WebKit QtAssignee: Siraj razick <siraj>
Status: RESOLVED FIXED    
Severity: Minor CC: hausmann, pierre-luc.beaudoin
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Expose boundingBox value in QWebHitTestResult hausmann: review+

Description Siraj razick 2008-06-03 07:39:34 PDT
QWebHitTestResult does not have an API function to get the boundingBox value from WebCore, I will attach a patch will that will expose the API function.
Comment 1 Siraj razick 2008-06-03 08:19:30 PDT
Created attachment 21477 [details]
Expose boundingBox value in QWebHitTestResult

Add API function QRect QWebHitResult::boundingBox() const
Comment 2 Siraj razick 2008-06-08 05:55:54 PDT
Comment on attachment 21477 [details]
Expose boundingBox value in QWebHitTestResult

Making obsolete till API documentation is added to the patch
Comment 3 Simon Hausmann 2008-10-21 15:30:36 PDT
Comment on attachment 21477 [details]
Expose boundingBox value in QWebHitTestResult

The patch looks good actually, it looks like we completly overlooked it :). Ariya added some API docs and I'm going to land it soon
Comment 4 Simon Hausmann 2008-10-21 15:33:31 PDT
Landed in r37764
Comment 5 Pierre-Luc Beaudoin 2008-10-21 15:54:53 PDT
I think you didn't overlook it, it wasn't "r?" :)
Comment 6 Siraj razick 2008-10-21 16:05:18 PDT
(In reply to comment #3)
> (From update of attachment 21477 [details] [edit])
> The patch looks good actually, it looks like we completly overlooked it :).
> Ariya added some API docs and I'm going to land it soon
> 
Thank you for the review + !and sorry for not adding the API docs. And Thank you Ariya for cleaning up the mess :) .
Comment 7 Simon Hausmann 2008-10-22 12:58:28 PDT
After a brief discussion we committed also r37787 that renames boundingBox() to boundingRect(), for consistency with the rest of the Qt API