Set the WebViewDelegate earlier during WebView creation to ensure that...

webkit/glue/webview_impl.cc

 /*
 * Copyright 2007 Google Inc. All Rights Reserved.
 *
 * Portions Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
 *
 * ***** BEGIN LICENSE BLOCK *****
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions
 * are met:
@@ skipping 295 matching lines @@
 
 // WebView ----------------------------------------------------------------
 
 /*static*/
 WebView* WebView::Create(WebViewDelegate* delegate,
                          const WebPreferences& prefs) {
   WebViewImpl* instance = new WebViewImpl();
   instance->AddRef();
   instance->SetPreferences(prefs);
 
-  // Here, we construct a new WebFrameImpl with a reference count of 0.  That
-  // is bumped up to 1 by InitMainFrame.  The reference count is decremented
-  // when the corresponding WebCore::Frame object is destroyed.
-  WebFrameImpl* main_frame = new WebFrameImpl();
+  // NOTE: The WebFrameImpl takes a reference to itself within InitMainFrame
+  // and releases that reference once the corresponding Frame is destroyed.
+  scoped_refptr<WebFrameImpl> main_frame = new WebFrameImpl();
+
+  // Set the delegate before initializing the frame, so that notifications like
+  // DidCreateDataSource make their way to the client.
+  instance->delegate_ = delegate;
   main_frame->InitMainFrame(instance);
 
-  // Set the delegate after initializing the main frame, to avoid trying to
-  // respond to notifications before we're fully initialized.
-  instance->delegate_ = delegate;
-
   WebDevToolsAgentDelegate* tools_delegate =
       delegate->GetWebDevToolsAgentDelegate();
   if (tools_delegate) {
     instance->devtools_agent_.reset(
       new WebDevToolsAgentImpl(instance, tools_delegate));
   }
 
   // Restrict the access to the local file system
   // (see WebView.mm WebView::_commonInitializationWithFrameName).
   FrameLoader::setLocalLoadPolicy(
@@ skipping 1605 matching lines @@
 
   return document->focusedNode();
 }
 
 HitTestResult WebViewImpl::HitTestResultForWindowPos(const IntPoint& pos) {
   IntPoint doc_point(
       page_->mainFrame()->view()->windowToContents(pos));
   return page_->mainFrame()->eventHandler()->
       hitTestResultAtPoint(doc_point, false);
 }