Summary: | [BlackBerry] Handle testRunner.setWillSendRequestReturnsNull() in DRT | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, rwlbuis, xan.lopez | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Carlos Garcia Campos
2013-06-20 05:14:40 PDT
Created attachment 205074 [details]
Patch
Comment on attachment 205074 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205074&action=review > Tools/DumpRenderTree/blackberry/DumpRenderTree.cpp:630 > + } Could this method just return bool? Is it possible for request to be null coming in? Also if the above two blocks can be combined you don't need the return. Created attachment 205097 [details]
Updated patch
Comment on attachment 205097 [details]
Updated patch
LGTM.
Comment on attachment 205097 [details] Updated patch Clearing flags on attachment: 205097 Committed r151780: <http://trac.webkit.org/changeset/151780> All reviewed patches have been landed. Closing bug. |