Avoid a crash in ChromeFrame in the BindToStorage call initiated when our active document is loaded. The crash occurs while dereferencing a

NULL delegate which is the case in the pending request object created by the active document to handle the initial load. 

Fix for this is to maintain a pending state in the request object. We ignore all OnProgress notifications in this state. When Chrome requests
data for the top level url this state is cleared.

Consolidated the number of bind context information structures into one which contains everything we need in ChromeFrame, i.e. to decide to
switch to Chrome, indicating whether the request came from Chrome, etc.

Review URL: http://codereview.chromium.org/1654012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44604 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 45 insertions, 0 deletions

diff --git a/chrome_frame/bind_context_info.cc b/chrome_frame/bind_context_info.cc
new file mode 100644
index 0000000..3eb2554
--- /dev/null
+++ b/chrome_frame/bind_context_info.cc
@@ -0,0 +1,45 @@
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome_frame/bind_context_info.h"
+#include "chrome_frame/utils.h"
+
+// This is non const due to API expectations
+static wchar_t* kBindContextInfo = L"_CHROMEFRAME_BIND_CONTEXT_INFO_";
+
+// BindContextInfo member definitions.
+BindContextInfo::BindContextInfo()
+    : no_cache_(false),
+      chrome_request_(false),
+      is_switching_(false) {
+}
+
+BindContextInfo* BindContextInfo::FromBindContext(IBindCtx* bind_context) {
+  if (!bind_context) {
+    NOTREACHED();
+    return NULL;
+  }
+
+  ScopedComPtr<IUnknown> context;
+  bind_context->GetObjectParam(kBindContextInfo, context.Receive());
+  if (context) {
+    return static_cast<BindContextInfo*>(context.get());
+  }
+
+  CComObject<BindContextInfo>* bind_context_info = NULL;
+  CComObject<BindContextInfo>::CreateInstance(&bind_context_info);
+  DCHECK(bind_context_info != NULL);
+
+  bind_context->RegisterObjectParam(kBindContextInfo, bind_context_info);
+  return bind_context_info;
+}
+
+void BindContextInfo::SetToSwitch(IStream* cache) {
+  is_switching_ = true;
+  if  (!no_cache_) {
+    cache_ = cache;
+    RewindStream(cache_);
+  }
+}
+