Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5684)

Unified Diff: chrome_frame/bho.cc

Issue 1589013: Switch renderer in Moniker patch... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome_frame/bho.cc
===================================================================
--- chrome_frame/bho.cc (revision 43957)
+++ chrome_frame/bho.cc (working copy)
@@ -251,16 +251,6 @@
}
}
}
- } else if (done) {
- if (!CheckForCFNavigation(browser, false)) {
- DLOG(INFO) << "Releasing cached data.";
- NavigationManager* mgr = NavigationManager::GetThreadInstance();
- if (mgr)
- mgr->ReleaseRequestData();
- } else {
- DLOG(INFO) << __FUNCTION__
- << " not freeing request data - browser tagged";
- }
}
return original_httpequiv(browser, shell_view, done, in_arg, out_arg);
@@ -326,18 +316,17 @@
ProtocolPatchMethod patch_method =
static_cast<ProtocolPatchMethod>(
GetConfigInt(PATCH_METHOD_IBROWSER, kPatchProtocols));
-
if (patch_method == PATCH_METHOD_INET_PROTOCOL) {
ProtocolSinkWrap::PatchProtocolHandlers();
state_ = PATCH_PROTOCOL;
+ } else if (patch_method == PATCH_METHOD_IBROWSER) {
+ state_ = PATCH_IBROWSER;
} else {
- DCHECK(patch_method == PATCH_METHOD_IBROWSER ||
- patch_method == PATCH_METHOD_IBROWSER_AND_MONIKER);
- state_ = PATCH_IBROWSER;
- if (patch_method == PATCH_METHOD_IBROWSER_AND_MONIKER) {
- MonikerPatch::Initialize();
- }
+ DCHECK(patch_method == PATCH_METHOD_MONIKER);
+ state_ = PATCH_MONIKER;
+ MonikerPatch::Initialize();
}
+
ret = true;
}

Powered by Google App Engine
This is Rietveld 408576698