Do not wrap non-HTTP(s) requests.

chrome_frame/protocol_sink_wrap.cc

 // Copyright (c) 2009 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 <htiframe.h>
 #include <mshtml.h>
 
 #include "chrome_frame/protocol_sink_wrap.h"
 
 #include "base/logging.h"
@@ skipping 127 matching lines @@
   if (bind_ctx_string) {
     IBindCtx* pbc = reinterpret_cast<IBindCtx*>(StringToInt(bind_ctx_string));
     bind_ctx.Attach(pbc);
     CoTaskMemFree(bind_ctx_string);
   }
 
   return bind_ctx;
 }
 
 bool ShouldWrapSink(IInternetProtocolSink* sink, const wchar_t* url) {
-  // TODO(stoyan): check the url scheme for http/https.
+  // Ignore everything that does not start with http:// or https://.
+  // |url| is already normalized (i.e. no leading spaces, capital letters in
+  // protocol etc) and non-null (we check in Hook_Start).
+  DCHECK(url != NULL);
+  if ((url != StrStrW(url, L"http://")) && (url != StrStrW(url, L"https://")))

[tommi (sloooow) - chröme, 2010/07/14 19:18:44]

use StartsWith?

[stoyan, 2010/07/14 19:34:10]

StartsWith wants me to create std::string object.
And apparently there is not StringPieceW::starts_with()

+    return false;
+
   ScopedComPtr<IHttpNegotiate> http_negotiate;
   HRESULT hr = DoQueryService(GUID_NULL, sink, http_negotiate.Receive());
   if (http_negotiate && !IsSubFrameRequest(http_negotiate))
     return true;
 
   return false;
 }
 
 // High level helpers
 bool IsCFRequest(IBindCtx* pbc) {
@@ skipping 497 matching lines @@
     // Explicit release, otherwise ~CComObjectStackEx will complain about
     // outstanding reference to us, because it runs before ~FakeProtocol
     prot.transaction_.Release();
   }
 }
 
 void TransactionHooks::RevertHooks() {
   vtable_patch::UnpatchInterfaceMethods(CTransaction_PatchInfo);
   vtable_patch::UnpatchInterfaceMethods(CTransaction2_PatchInfo);
 }