[Webkit-unassigned] [Bug 114944] [BlackBerry] Crash due to an assert running test editing/execCommand/indent-paragraphs.html

[bugzilla-daemon at webkit.org]

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


Rob Buis <rwlbuis at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #199009|review?                     |review-
               Flag|                            |




--- Comment #2 from Rob Buis <rwlbuis at gmail.com>  2013-05-13 08:54:57 PST ---
(From update of attachment 199009)
View in context: https://bugs.webkit.org/attachment.cgi?id=199009&action=review

Looks great, but the new early exit needs to be verified and at least an error/log has to be there.

> Source/WebCore/platform/network/blackberry/ResourceHandleBlackBerry.cpp:160
> +    if (!NetworkManager::instance()->startJob(playerId, handle, frame, defersLoading))

It would be nice to set the error param here.

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