Added a new policy setting as well as corrisponding registry key to allow GCF users the option to d…

chrome_frame/protocol_sink_wrap.cc

 // Copyright (c) 2011 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 <algorithm>
 
 #include "chrome_frame/protocol_sink_wrap.h"
 
 #include "base/logging.h"
 #include "base/memory/singleton.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/win/scoped_bstr.h"
 #include "chrome_frame/bho.h"
 #include "chrome_frame/bind_context_info.h"
 #include "chrome_frame/exception_barrier.h"
 #include "chrome_frame/function_stub.h"
 #include "chrome_frame/policy_settings.h"
 #include "chrome_frame/utils.h"
 
+using std::min;
+
 // BINDSTATUS_SERVER_MIMETYPEAVAILABLE == 54. Introduced in IE 8, so
 // not in everyone's headers yet. See:
 // http://msdn.microsoft.com/en-us/library/ms775133(VS.85,loband).aspx
 #ifndef BINDSTATUS_SERVER_MIMETYPEAVAILABLE
 #define BINDSTATUS_SERVER_MIMETYPEAVAILABLE 54
 #endif
 
 bool ProtocolSinkWrap::ignore_xua_ = false;
 
 static const char kTextHtmlMimeType[] = "text/html";
@@ skipping 424 matching lines @@
       return S_OK;
   }
 
   // We are just pass through at this point, avoid false positive crash reports.
   ExceptionBarrierReportOnlyModule barrier;
   return delegate->ReportProgress(status_code, status_text);
 }
 
 HRESULT ProtData::ReportData(IInternetProtocolSink* delegate,
                               DWORD flags, ULONG progress, ULONG max_progress) {
+  bool ignore_Use_GCF_MetaData = SkipMetaDataCheck();

[grt (UTC plus 2), 2013/08/02 01:42:50]

since SkipMetaDataCheck may do work (hit the registry), move this down so that
it's only called when it is needed. for example, on line 485:
  renderer_type_ = SkipMetadataCheck() ? RENDERER_TYPE_OTHER :
      DetermineRendererType(buffer_, buffer_size_, last_chance);

   if (renderer_type_ != RENDERER_TYPE_UNDETERMINED) {
     // We are just pass through now, avoid false positive crash reports.
     ExceptionBarrierReportOnlyModule barrier;
     return delegate->ReportData(flags, progress, max_progress);
   }
 
   HRESULT hr = FillBuffer();
 
   bool last_chance = false;
   if (hr == S_OK || hr == S_FALSE) {
     last_chance = true;
   }
 
-  renderer_type_ = DetermineRendererType(buffer_, buffer_size_, last_chance);
+  renderer_type_ = ignore_Use_GCF_MetaData ? RENDERER_TYPE_OTHER
+      : DetermineRendererType(buffer_, buffer_size_, last_chance);
 
   if (renderer_type_ == RENDERER_TYPE_UNDETERMINED) {
     // do not report anything, we need more data.
     return S_OK;
   }
 
   if (IsChrome(renderer_type_)) {
     DVLOG(1) << "Forwarding BINDSTATUS_MIMETYPEAVAILABLE " << kChromeMimeType;
     SaveReferrer(delegate);
     delegate->ReportProgress(BINDSTATUS_MIMETYPEAVAILABLE, kChromeMimeType);
@@ skipping 505 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);
 }