gpu: GPU service scheduler.

gpu/command_buffer/common/gpu_stream_constants.h

Index: gpu/command_buffer/common/gpu_stream_constants.h
diff --git a/gpu/ipc/common/gpu_stream_constants.h b/gpu/command_buffer/common/gpu_stream_constants.h
similarity index 51%
rename from gpu/ipc/common/gpu_stream_constants.h
rename to gpu/command_buffer/common/gpu_stream_constants.h
index abab39895328ca4a1b09e0726aed23fc02e913d4..7e6fe438dc86c489c3488df9098b59c6b15ede70 100644
--- a/gpu/ipc/common/gpu_stream_constants.h
+++ b/gpu/command_buffer/common/gpu_stream_constants.h
@@ -5,11 +5,20 @@
 #ifndef GPU_IPC_COMMON_GPU_STREAM_CONSTANTS_H_
 #define GPU_IPC_COMMON_GPU_STREAM_CONSTANTS_H_
 
+#include "gpu/gpu_export.h"
+
 namespace gpu {
 
-enum class GpuStreamPriority { REAL_TIME, HIGH, NORMAL, LOW, LAST = LOW };
+enum class GpuStreamPriority : int { REAL_TIME, HIGH, NORMAL, LOW, LAST = LOW };
+static const int kNumGpuStreamPriorities = 4;
+
+enum GpuStreamId {
+  GPU_STREAM_INVALID = -1,
+  GPU_STREAM_DEFAULT = 0,
+  GPU_STREAM_WORKER = 1,

[piman, 2017/05/09 00:48:38]

I'm not excited about defining here a notion that belongs to the higher levels
(worker). For example we may decide in the future that we want a separate stream
for Pepper or WebGL or video, but those are concepts that only exist in content/

I would prefer keeping that enum in content/, if we want a static allocation of
stream ids. Otherwise the previous dynamic allocation doesn't seem all that
bothersome?

[sunnyps, 2017/05/10 23:15:15]

The service side implementation of stream lifetimes is simpler for static
allocation of streams. The service doesn't need to specifically know the stream
ids so I've moved those to content/common/gpu_stream_constants.h.

+};
 
-enum GpuStreamId { GPU_STREAM_DEFAULT = -1, GPU_STREAM_INVALID = 0 };
+GPU_EXPORT const char* GpuStreamPriorityToString(GpuStreamPriority priority);
 
 }  // namespace gpu