Move attributes and context type out to ContextProviderCommandBuffer.

content/common/gpu/client/command_buffer_metrics.h

Index: content/common/gpu/client/command_buffer_metrics.h
diff --git a/content/common/gpu/client/command_buffer_metrics.h b/content/common/gpu/client/command_buffer_metrics.h
index 0cd62bb22096529427e985aa88433ce05ef88203..f354f5c47d7b9bc3dbccf455102ca587f52af7a5 100644
--- a/content/common/gpu/client/command_buffer_metrics.h
+++ b/content/common/gpu/client/command_buffer_metrics.h
@@ -10,8 +10,9 @@
 #include "gpu/command_buffer/common/constants.h"
 
 namespace content {
+namespace command_buffer_metrics {

[danakj, 2016/04/29 23:47:19]

I put this file into a namespace cuz it was hard to find these functions, they
look like member methods or local statics or something. And the style guide says
to use a namespace not a class to do this.

 
-enum CommandBufferContextType {
+enum ContextType {
   DISPLAY_COMPOSITOR_ONSCREEN_CONTEXT,
   BROWSER_OFFSCREEN_MAINTHREAD_CONTEXT,
   BROWSER_WORKER_CONTEXT,
@@ -25,14 +26,15 @@ enum CommandBufferContextType {
   OFFSCREEN_CONTEXT_FOR_TESTING = CONTEXT_TYPE_UNKNOWN,
 };
 
-std::string CommandBufferContextTypeToString(CommandBufferContextType type);
+std::string ContextTypeToString(ContextType type);
 
-void UmaRecordContextInitFailed(CommandBufferContextType type);
+void UmaRecordContextInitFailed(ContextType type);
 
-void UmaRecordContextLost(CommandBufferContextType type,
+void UmaRecordContextLost(ContextType type,
                           gpu::error::Error error,
                           gpu::error::ContextLostReason reason);
 
+}  // namespace command_buffer_metrics
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_CLIENT_COMMAND_BUFFER_METRICS_H_