Global Resource Coordinator: Basic service internals

services/resource_coordinator/public/cpp/coordination_unit_events.h

Index: services/resource_coordinator/public/cpp/coordination_unit_events.h
diff --git a/services/resource_coordinator/public/cpp/coordination_unit_events.h b/services/resource_coordinator/public/cpp/coordination_unit_events.h
new file mode 100644
index 0000000000000000000000000000000000000000..214aa8db9383f997277f4c35a2f0695b4586a6b8
--- /dev/null
+++ b/services/resource_coordinator/public/cpp/coordination_unit_events.h
@@ -0,0 +1,23 @@
+// 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_PUBLIC_CPP_EVENTS_H_

[Primiano Tucci (use gerrit), 2017/04/06 18:09:46]

nit: macro doesn't match the file name

[oystein (OOO til 10th of July), 2017/04/10 20:03:00]

Done.

+#define SERVICES_RESOURCE_COORDINATOR_PUBLIC_CPP_EVENTS_H_
+
+namespace resource_coordinator {
+
+enum EventType : uint32_t {

[Primiano Tucci (use gerrit), 2017/04/06 18:09:46]

shouldn't this be an enum class to reduce namespace pollution?
These enum names will end up in the general resource_coordinator namespace,
which means that you won't be able to use "NUM_EVENTS" or "TEST_EVENT" anywhere
else (in any other enum) if this is just an enum.

[oystein (OOO til 10th of July), 2017/04/10 20:03:00]

Oops, meant to do this. Fixed.

+  INVALID_EVENT,
+  TEST_EVENT,
+  ON_COMMIT,
+  ON_WEBCONTENTS_SHOWN,
+  ON_WEBCONTENTS_HIDDEN,
+  ON_RENDERER_FRAME_CREATED,
+  ON_PROCESS_AUDIO_STARTED,
+  ON_PROCESS_AUDIO_STOPPED,
+  NUM_EVENTS
+};
+
+}  // resource_coordinator
+
+#endif  // SERVICES_RESOURCE_COORDINATOR_PUBLIC_CPP_EVENTS_H_