DocumentThreadableLoader: Add guards for sync notifyFinished() in setResource()

In loadRequest(), setResource() can call clear() synchronously:
  DocumentThreadableLoader::clear()
  DocumentThreadableLoader::handleError()
  Resource::didAddClient()
  RawResource::didAddClient()
and thus |m_client| can be null while resource() isn't null after setResource(),
causing crashes (Issue 595964).

This CL checks whether |*this| is destructed and
whether |m_client| is null after setResource().

BUG=595964

Review-Url: https://codereview.chromium.org/1902683002
Cr-Commit-Position: refs/heads/master@{#391001}
void DocumentThreadableLoader::redirectBlocked()
{
    ThreadableLoaderClient* client = m_client;
    clear();
    client->didFailRedirectCheck();
}

void DocumentThreadableLoader::redirectBlocked()
{
    ThreadableLoaderClient* client = m_client;
    clear();
    client->didFailRedirectCheck();
}
