SubresourceLoader::didFail() should only log message if state is Initialized
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 23 Jun 2018 01:22:04 +0000 (01:22 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 23 Jun 2018 01:22:04 +0000 (01:22 +0000)
commita4c69765b1e001cbb7a447e2609fa1fc6f13f626
tree9c19b29f36857ee056b129ebee52c041f9d009e1
parent7c9580ef2651fc4974530f5ed9931f456ee6f89e
SubresourceLoader::didFail() should only log message if state is Initialized
https://bugs.webkit.org/show_bug.cgi?id=185124

Patch by Woodrow Wang <woodrow_wang@apple.com> on 2018-06-22
Reviewed by Daniel Bates.

Functionality does not change. Moved console logging to be
after checking state of subresource loader. We only need to
log if the state is initialized. This is consistent with other
functions in the file. We can also remove a null check for the
frame pointer (m_frame). The superclass ResourceLoader constructor
takes an lvalue reference and initializes m_frame, ensuring
m_frame cannot be null. It is only set to null by
ResourceLoader::releaseResources(), which is only called after the
resource finishes loading or fails to load. Thus, in didFail(),
m_frame must be non-null when we're logging and up until the end of the function.

* loader/SubresourceLoader.cpp:
(WebCore::SubresourceLoader::didFail):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233117 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/loader/SubresourceLoader.cpp