[Webkit-unassigned] [Bug 28802] New: Remove ASSERT(frame) in ResourceHandle::start (ResourceHandleCurl.cpp)

[bugzilla-daemon at webkit.org]

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

           Summary: Remove ASSERT(frame) in ResourceHandle::start
                    (ResourceHandleCurl.cpp)
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Other
        OS/Version: Other
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Platform
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: kwangyul.seo at gmail.com


bool ResourceHandle::start(Frame* frame)
{
    ASSERT(frame);
    ResourceHandleManager::sharedInstance()->add(this);
    return true;
}

Remove ASSERT in the first line because the frame could be null if the
ResourceHandle is not associated with any frame, e.g. if we are downloading a
file.

ResourceHandleSoup.cpp handles this case. Follow what ResourceHandleSoup.cpp
does.

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