[webkit-reviews] review granted: [Bug 75611] [BlackBerry] Use the concrete FrameNetworkingContextBlackBerry to access Frame : [Attachment 121240] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 6 13:36:48 PST 2012


Antonio Gomes <tonikitoo at webkit.org> has granted Leo Yang
<leo.yang at torchmobile.com.cn>'s request for review:
Bug 75611: [BlackBerry] Use the concrete FrameNetworkingContextBlackBerry to
access Frame
https://bugs.webkit.org/show_bug.cgi?id=75611

Attachment 121240: Patch
https://bugs.webkit.org/attachment.cgi?id=121240&action=review

------- Additional Comments from Antonio Gomes <tonikitoo at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=121240&action=review


> Source/WebCore/ChangeLog:11
> +	   networking context. NetworkingContext::wrappedFrame() was added by
the BlackBerry
> +	   porting internally. We don't want to upstream this specific change,
so use

You could mention why it was added and why it is not needed or it is fine to
use the "concrete" one?


More information about the webkit-reviews mailing list