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

Unified Diff: content/browser/dom_storage/dom_storage_namespace.cc

Issue 126033004: Clean up ChildProcessHost unique id generation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: mac Created 6 years, 11 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
« no previous file with comments | « chrome/browser/signin/signin_manager.cc ('k') | content/common/child_process_host_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/dom_storage/dom_storage_namespace.cc
diff --git a/content/browser/dom_storage/dom_storage_namespace.cc b/content/browser/dom_storage/dom_storage_namespace.cc
index ac1e2f89cd0f5c372e30f8025ca512233636ef95..d36147f076e474a6b5ba7fdb7253eefae39d8dc8 100644
--- a/content/browser/dom_storage/dom_storage_namespace.cc
+++ b/content/browser/dom_storage/dom_storage_namespace.cc
@@ -16,8 +16,8 @@
#include "content/browser/dom_storage/dom_storage_context_impl.h"
#include "content/browser/dom_storage/dom_storage_task_runner.h"
#include "content/browser/dom_storage/session_storage_database.h"
-#include "content/common/child_process_host_impl.h"
#include "content/common/dom_storage/dom_storage_types.h"
+#include "content/public/common/child_process_host.h"
namespace content {
@@ -247,14 +247,14 @@ DOMStorageNamespace::GetAreaHolder(const GURL& origin) {
}
void DOMStorageNamespace::AddTransactionLogProcessId(int process_id) {
- DCHECK(process_id != ChildProcessHostImpl::kInvalidChildProcessId);
+ DCHECK(process_id != ChildProcessHost::kInvalidUniqueID);
DCHECK(transactions_.count(process_id) == 0);
TransactionData* transaction_data = new TransactionData;
transactions_[process_id] = transaction_data;
}
void DOMStorageNamespace::RemoveTransactionLogProcessId(int process_id) {
- DCHECK(process_id != ChildProcessHostImpl::kInvalidChildProcessId);
+ DCHECK(process_id != ChildProcessHost::kInvalidUniqueID);
DCHECK(transactions_.count(process_id) == 1);
delete transactions_[process_id];
transactions_.erase(process_id);
@@ -353,7 +353,7 @@ SessionStorageNamespace::MergeResult DOMStorageNamespace::Merge(
}
bool DOMStorageNamespace::IsLoggingRenderer(int process_id) {
- DCHECK(process_id != ChildProcessHostImpl::kInvalidChildProcessId);
+ DCHECK(process_id != ChildProcessHost::kInvalidUniqueID);
if (transactions_.count(process_id) < 1)
return false;
return !transactions_[process_id]->max_log_size_exceeded;
« no previous file with comments | « chrome/browser/signin/signin_manager.cc ('k') | content/common/child_process_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698