cc: Create cc::DebugColors class to hold all the debugging color/width constants.

cc/debug_colors.h

+// Copyright 2012 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 CC_DEBUG_COLORS_H_
+#define CC_DEBUG_COLORS_H_
+
+#include "third_party/skia/include/core/SkColor.h"
+
+namespace cc {
+
+class LayerTreeHostImpl;
+
+class DebugColors {
+ public:
+  static SkColor kTiledContentLayerBorderColor();

[tfarina, 2012/11/15 21:06:10]

was the use of 'k' prefix intentional?

[danakj, 2012/11/15 21:11:34]

oh.. i had made them constants before. I'll fix the names, thanks.

+  static int     kTiledContentLayerBorderWidth(const LayerTreeHostImpl* hostImpl);

[tfarina, 2012/11/15 21:06:10]

if this were to follow chromium style then three minor points:

1- only one space after 'int'
2- s/hostImpl/host_impl
3- respect the 80 char columns.

+
+  static SkColor kContentLayerBorderColor();
+  static int     kContentLayerBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kMaskingLayerBorderColor();
+  static int     kMaskingLayerBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kContainerLayerBorderColor();
+  static int     kContainerLayerBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kSurfaceBorderColor();
+  static SkColor kSurfaceBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kSurfaceReplicaBorderColor();
+  static SkColor kSurfaceReplicaBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kTileBorderColor();
+  static int     kTileBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kMissingTileBorderColor();
+  static int     kMissingTileBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kCulledTileBorderColor();
+  static int     kCulledTileBorderWidth(const LayerTreeHostImpl* hostImpl);
+
+  static SkColor kEvictedTileCheckerboardColor();
+  static SkColor kInvalidatedTileCheckerboardColor();
+};

[tfarina, 2012/11/15 21:06:10]

DISALLOW_IMPLICIT_CONSTRUCTORS

[danakj, 2012/11/15 21:11:34]

ah.. fancy.

+
+}  // namespace cc
+
+#endif  // CC_DEBUG_COLORS_H_