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

Unified Diff: content/browser/frame_host/render_frame_host_manager.cc

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years 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: content/browser/frame_host/render_frame_host_manager.cc
diff --git a/content/browser/frame_host/render_frame_host_manager.cc b/content/browser/frame_host/render_frame_host_manager.cc
index 59cbe436c80a945cff6f16a1e98cea75ad0b679d..a95162e4deb05ba4e681f2481aa1a9edba8dbf29 100644
--- a/content/browser/frame_host/render_frame_host_manager.cc
+++ b/content/browser/frame_host/render_frame_host_manager.cc
@@ -1059,8 +1059,8 @@ RenderFrameProxyHost* RenderFrameHostManager::CreateRenderFrameProxyHost(
SiteInstance* site_instance,
RenderViewHostImpl* rvh) {
int site_instance_id = site_instance->GetId();
- CHECK(proxy_hosts_.find(site_instance_id) == proxy_hosts_.end())
- << "A proxy already existed for this SiteInstance.";
+ // A proxy already existed for this SiteInstance.
+ CHECK(proxy_hosts_.find(site_instance_id) == proxy_hosts_.end());
RenderFrameProxyHost* proxy_host =
new RenderFrameProxyHost(site_instance, rvh, frame_tree_node_);
proxy_hosts_[site_instance_id] = base::WrapUnique(proxy_host);

Powered by Google App Engine
This is Rietveld 408576698