Move renderFallbackContent() into FrameOwner interface.

Source/core/loader/DocumentLoader.cpp

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 440 matching lines @@
         m_mainResource->setDataBufferingPolicy(BufferData);
 
     if (!shouldContinueForResponse()) {
         InspectorInstrumentation::continueWithPolicyIgnore(m_frame, this, m_mainResource->identifier(), m_response);
         cancelMainResourceLoad(ResourceError::cancelledError(m_request.url()));
         return;
     }
 
     if (m_response.isHTTP()) {
         int status = m_response.httpStatusCode();
-        // FIXME: Fallback content only works if the parent is in the same processs.
         if ((status < 200 || status >= 300) && m_frame->owner()) {
-            if (!m_frame->deprecatedLocalOwner()) {
-                ASSERT_NOT_REACHED();
-            } else if (m_frame->deprecatedLocalOwner()->isObjectElement()) {
-                m_frame->deprecatedLocalOwner()->renderFallbackContent();
-                // object elements are no longer rendered after we fallback, so don't
-                // keep trying to process data from their load
-                cancelMainResourceLoad(ResourceError::cancelledError(m_request.url()));
+            if (m_frame->owner()->renderFallbackContent()) {

[Nate Chapin, 2015/05/07 18:01:07]

This block looks empty? Is returning a bool actually necesssary?

[dcheng, 2015/05/07 18:02:12]

I think this is already cleaned up in the latest PS =)

             }
         }
     }
 }
 
 void DocumentLoader::ensureWriter(const AtomicString& mimeType, const KURL& overridingURL)
 {
     if (m_writer)
         return;
 
@@ skipping 304 matching lines @@
 // This is only called by FrameLoader::replaceDocumentWhileExecutingJavaScriptURL()
 void DocumentLoader::replaceDocumentWhileExecutingJavaScriptURL(const DocumentInit& init, const String& source, Document* ownerDocument)
 {
     m_writer = createWriterFor(ownerDocument, init, mimeType(), m_writer ? m_writer->encoding() : emptyAtom, true, ForceSynchronousParsing);
     if (!source.isNull())
         m_writer->appendReplacingData(source);
     endWriting(m_writer.get());
 }
 
 } // namespace blink