Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(474)

Unified Diff: webrtc/modules/desktop_capture/color.h

Issue 2268093002: [WebRTC] A real ScreenCapturer test (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Resolve review comments Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..ade6f0731c37a794c4f899fd2c948474958dacd0
--- /dev/null
+++ b/webrtc/modules/desktop_capture/color.h
@@ -0,0 +1,70 @@
+/*
+ * 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, which can be converted
+// from and to uint32_t format. This structure also provides functions to
+// compare with uint8_t array, say, DesktopFrame::data(). When converting from
+// or to uint32_t format, it's big-endian and little-endian safe, but 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;
+
+ // Converts current Color into its uint format in BGRA order (little-endian)
+ // or ARGB order (big-endian) for a quick assignment or compare with
+ // DesktopFrame::data().
+ uint32_t ToUInt32() const;
Jamie 2016/08/31 17:39:39 I don't think you're using this method. Please rem
Hzj_jie 2016/08/31 21:22:39 This will be used in change 2202443002. I can sure
+
+ uint8_t blue() const;
+
+ uint8_t green() const;
+
+ uint8_t red() const;
+
+ uint8_t alpha() const;
+
+ uint8_t bgra[DesktopFrame::kBytesPerPixel];
+};
+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_

Powered by Google App Engine
This is Rietveld 408576698