GitLab

This was made non-refcounted class in https://crrev.com/1190133002,
but looks like this could cause crashes in some corner cases.

To be more specific:
FrameLoaderClientImpl::dispatchDidFinishDocumentLoad() calls
WebFrameClient callback first and then WebView callback next, but
if the loader is deleted in the first callback an attempt to call the
second callback on a member variable would crash.

We could probably avoid this crash without making the loader class
ref-counted again, but for now I would like to fix the crash by simply
reverting some of the changes that're causing the crash.

BUG=524694

Review URL: https://codereview.chromium.org/1322523002

git-svn-id: svn://svn.chromium.org/blink/trunk@201346 bbb929c8-8fbe-4397-9dbb-9b2b20218538