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

Unified Diff: services/resource_coordinator/memory/coordinator/process_map.h

Issue 2895083004: memory-infra: rename service folder to memory_instrumentation (Closed)
Patch Set: comments Created 3 years, 7 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: services/resource_coordinator/memory/coordinator/process_map.h
diff --git a/services/resource_coordinator/memory/coordinator/process_map.h b/services/resource_coordinator/memory/coordinator/process_map.h
deleted file mode 100644
index e68a4a5091b5dc883b31f845b1ba738a637f2246..0000000000000000000000000000000000000000
--- a/services/resource_coordinator/memory/coordinator/process_map.h
+++ /dev/null
@@ -1,58 +0,0 @@
-// Copyright 2017 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.
-
-#ifndef SERVICES_RESOURCE_COORDINATOR_MEMORY_COORDINATOR_PROCESS_MAP_H_
-#define SERVICES_RESOURCE_COORDINATOR_MEMORY_COORDINATOR_PROCESS_MAP_H_
-
-#include <map>
-#include <vector>
-
-#include "base/gtest_prod_util.h"
-#include "base/process/process_handle.h"
-#include "mojo/public/cpp/bindings/binding.h"
-#include "services/service_manager/public/cpp/identity.h"
-#include "services/service_manager/public/interfaces/service_manager.mojom.h"
-
-namespace service_manager {
-class Connector;
-} // namespace service_manager
-
-namespace memory_instrumentation {
-
-// Maintains a map from service_manager::Identity to base::ProcessId by
-// listening for connections. This allows |pid| lookup by
-// service_manager::Identity. The assumption is that all the processes that will
-// register, via their client libraries, to the memory instrumentation service
-// will also necessarily connect to the service manager.
-class ProcessMap : public service_manager::mojom::ServiceManagerListener {
- public:
- explicit ProcessMap(service_manager::Connector* connector);
- ~ProcessMap() override;
-
- // Returns the pid for a client given its identity.
- base::ProcessId GetProcessId(const service_manager::Identity&) const;
-
- protected:
- private:
- FRIEND_TEST_ALL_PREFIXES(ProcessMapTest, TypicalCase);
- FRIEND_TEST_ALL_PREFIXES(ProcessMapTest, PresentInInit);
-
- using Identity = service_manager::Identity;
- using RunningServiceInfoPtr = service_manager::mojom::RunningServiceInfoPtr;
-
- // Overridden from service_manager::mojom::ServiceManagerListener.
- void OnInit(std::vector<RunningServiceInfoPtr> instances) override;
- void OnServiceCreated(RunningServiceInfoPtr instance) override;
- void OnServiceStarted(const Identity&, uint32_t pid) override;
- void OnServiceFailedToStart(const Identity&) override;
- void OnServiceStopped(const Identity&) override;
-
- mojo::Binding<service_manager::mojom::ServiceManagerListener> binding_;
- std::map<Identity, base::ProcessId> instances_;
-
- DISALLOW_COPY_AND_ASSIGN(ProcessMap);
-};
-
-} // namespace memory_instrumentation
-#endif // SERVICES_RESOURCE_COORDINATOR_MEMORY_COORDINATOR_PROCESS_MAP_H_

Powered by Google App Engine
This is Rietveld 408576698