Git commit b25d506ed4938eb858654549f0208a4948505fe8 by Maks Orlovich.
Committed on 28/05/2011 at 17:01.
Pushed by orlovich into branch 'KDE/4.7'.

Don't ask KHTMLRun to handle embedding cases with onlyLocalReferences, we want to
swallow them ourselves given what the mode is for.

Fixes what #248478 turned to (original issue got fixed organically, follow up
comments are technically unrelated)

+ Remove unused return and indent inconsistency while I am at it

BUG: 248478

M  +15   -16   khtml/khtml_part.cpp     
M  +1    -1    khtml/khtml_part.h     

http://commits.kde.org/kdelibs/b25d506ed4938eb858654549f0208a4948505fe8

diff --git a/khtml/khtml_part.cpp b/khtml/khtml_part.cpp
index ec89b0c..817e606 100644
--- a/khtml/khtml_part.cpp
+++ b/khtml/khtml_part.cpp
@@ -4131,7 +4131,7 @@ KParts::ScriptableExtension *KHTMLPart::scriptableExtension( const DOM::NodeImpl
     return 0L;
 }
 
-bool KHTMLPart::loadFrameElement( DOM::HTMLPartContainerElementImpl *frame, const QString &url,
+void KHTMLPart::loadFrameElement( DOM::HTMLPartContainerElementImpl *frame, const QString &url,
                                   const QString &frameName, const QStringList &params, bool isIFrame )
 {
     //kDebug( 6050 ) << this << " requestFrame( ..., " << url << ", " << frameName << " )";
@@ -4165,20 +4165,18 @@ bool KHTMLPart::loadFrameElement( DOM::HTMLPartContainerElementImpl *frame, cons
         // ### load event on the kid?
         navigateLocalProtocol(child, part, KUrl("about:blank"));
         connectToChildPart(child, part, "text/html" /* mimetype of the part, not what's being loaded */);
-  }
+    }
 
-  KUrl u = url.isEmpty() ? KUrl() : completeURL( url );
+    KUrl u = url.isEmpty() ? KUrl() : completeURL( url );
 
-  // Since we don't specify args here a KHTMLRun will be used to determine the
-  // mimetype, which will then be  passed down at the bottom of processObjectRequest
-  // inside URLArgs to the part. In our particular case, this means that we can
-  // use that inside KHTMLPart::openUrl to route things appropriately.
-  child->m_bCompleted = false;
-  if (!requestObject( child, u ) && !child->m_run) {
-      child->m_bCompleted = true;
-      return false;
-  }
-  return true;
+    // Since we don't specify args here a KHTMLRun will be used to determine the
+    // mimetype, which will then be  passed down at the bottom of processObjectRequest
+    // inside URLArgs to the part. In our particular case, this means that we can
+    // use that inside KHTMLPart::openUrl to route things appropriately.
+    child->m_bCompleted = false;
+    if (!requestObject( child, u ) && !child->m_run) {
+        child->m_bCompleted = true;
+    }
 }
 
 QString KHTMLPart::requestFrameName()
@@ -4299,11 +4297,12 @@ bool KHTMLPart::processObjectRequest( khtml::ChildFrame *child, const KUrl &_url
     // though -> the reference becomes invalid -> crash is likely
     KUrl url( _url );
 
-    // If we are not permitting anything remote, or khtmlrun called us with
-    // empty url + mimetype to indicate a loading error, we obviosuly failed
+    // khtmlrun called us with empty url + mimetype to indicate a loading error,
+    // we obviosuly failed; but we can return true here since we don't want it
+    // doing anything more, while childLoadFailure is enough to notify our kid.
     if ( d->m_onlyLocalReferences || ( url.isEmpty() && mimetype.isEmpty() ) ) {
         childLoadFailure(child);
-        return false;
+        return true;
     }
 
     // we also want to ignore any spurious requests due to closing when parser is being cleared. These should be
diff --git a/khtml/khtml_part.h b/khtml/khtml_part.h
index a0104e3..18d3a9a 100644
--- a/khtml/khtml_part.h
+++ b/khtml/khtml_part.h
@@ -1751,7 +1751,7 @@ private:
   QString requestFrameName();
 
   // Requests loading of a frame or iframe element
-  bool loadFrameElement( DOM::HTMLPartContainerElementImpl *frame, const QString &url, const QString &frameName,
+  void loadFrameElement( DOM::HTMLPartContainerElementImpl *frame, const QString &url, const QString &frameName,
                          const QStringList &args = QStringList(), bool isIFrame = false );
 
   // Requests loading of an object or embed element. Returns true if