[WebRTC] A real ScreenCapturer test

webrtc/modules/desktop_capture/color.h

Index: webrtc/modules/desktop_capture/color.h
diff --git a/webrtc/modules/desktop_capture/color.h b/webrtc/modules/desktop_capture/color.h
new file mode 100644
index 0000000000000000000000000000000000000000..472eb93099f4fec2cf677f2d2c621d221f4e8cd5
--- /dev/null
+++ b/webrtc/modules/desktop_capture/color.h
@@ -0,0 +1,63 @@
+/*
+ *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
+ *
+ *  Use of this source code is governed by a BSD-style license
+ *  that can be found in the LICENSE file in the root of the source
+ *  tree. An additional intellectual property rights grant can be found
+ *  in the file PATENTS.  All contributing project authors may
+ *  be found in the AUTHORS file in the root of the source tree.
+ */
+
+#ifndef WEBRTC_MODULES_DESKTOP_CAPTURE_COLOR_H_
+#define WEBRTC_MODULES_DESKTOP_CAPTURE_COLOR_H_
+
+#include <stdint.h>
+
+#include "webrtc/modules/desktop_capture/desktop_frame.h"
+
+namespace webrtc {
+
+// A four-byte structure to store a color in BGRA format. This structure also
+// provides functions to compare with uint8_t array, say, DesktopFrame::data().
+// It always uses BGRA order for internal storage to match DesktopFrame::data().
+//
+// This struct is for testing purpose only, and should not be used in production
+// logic.
+struct Color final {
+  // Creates a color with BGRA channels.
+  Color(uint8_t blue, uint8_t green, uint8_t red, uint8_t alpha);
+
+  // Creates a color with BGR channels, and set alpha channel to 255 (opaque).
+  Color(uint8_t blue, uint8_t green, uint8_t red);
+
+  // Returns true if |bgra| four-byte in BGRA order represents a same color as
+  // |this|.
+  bool operator==(const uint8_t* const bgra) const;
+
+  // Returns true if |bgra| four-byte in BGRA order represents a different color
+  // as |this|.
+  bool operator!=(const uint8_t* const bgra) const;
+
+  // Returns true if |this| and |right| is the same color.
+  bool operator==(const Color& right) const;
+
+  // Returns true if |this| and |right| are different colors.
+  bool operator!=(const Color& right) const;
+
+  uint8_t blue() const;

[Sergey Ulanov, 2016/09/01 19:26:38]

since these are lower-case accessors they can be inlined here.

[Hzj_jie, 2016/09/01 23:25:09]

Done.

+
+  uint8_t green() const;
+
+  uint8_t red() const;
+
+  uint8_t alpha() const;
+
+  uint8_t bgra[DesktopFrame::kBytesPerPixel];

[Sergey Ulanov, 2016/09/01 19:26:38]

do you really need this as an array? why not just 4 uint8_t fields and then you
wouldn't need the 4 methods above.

[Hzj_jie, 2016/09/01 23:25:09]

Done.

+};
+static_assert(
+    DesktopFrame::kBytesPerPixel == sizeof(uint32_t),
+    "A pixel in DesktopFrame should be safe to represent by a uint32_t");
+
+}  // namespace webrtc
+
+#endif  // WEBRTC_MODULES_DESKTOP_CAPTURE_COLOR_H_