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

Issue 273823002: Adding gtest helpers for BeginFrameArgs. (Closed)

Created:
6 years, 7 months ago by mithro-old
Modified:
6 years, 7 months ago
Reviewers:
brianderson
CC:
chromium-reviews, cc-bugs_chromium.org
Visibility:
Public.

Description

Adding gtest helpers for BeginFrameArgs. * Allow BeginFrameArgs inside an EXPECT_EQ call. EXPECT_EQ(args1, args2) * Allow gtest to pretty print BeginFrameArgs objects. Output use to look like; --------------------------------------------------- Expected: args1 Which is: 16 byte object <0xFF 0xFF 0xFF 0x01 ...> Actual: args2 Which is: 16 byte object <0xFF 0xFF 0xFF 0x01 ...> --------------------------------------------------- Output now looks like; --------------------------------------------------- Expected: args1 Which is: BeginFrameArgs(1, 2, 3) Actual: args2 Which is: BeginFrameArgs(1, 2, 4) --------------------------------------------------- BUG=371223 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271413

Patch Set 1 #

Total comments: 2

Patch Set 2 : Making the operator== use const. #

Patch Set 3 : Rebasing #

Patch Set 4 : Adding unittest. #

Patch Set 5 : Rebase. #

Patch Set 6 : Rebasing onto master. #

Patch Set 7 : Change EXPECT_EQ(true/false to EXPECT_TRUE/FALSE(. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+114 lines, -5 lines) Patch
M cc/cc_tests.gyp View 1 2 3 4 5 1 chunk +1 line, -0 lines 0 comments Download
M cc/output/begin_frame_args.h View 1 2 3 4 5 1 chunk +2 lines, -1 line 0 comments Download
M cc/output/begin_frame_args.cc View 1 2 3 4 5 1 chunk +5 lines, -4 lines 0 comments Download
A cc/output/begin_frame_args_unittest.cc View 1 2 3 4 5 6 1 chunk +77 lines, -0 lines 0 comments Download
M cc/test/begin_frame_args_test.h View 1 2 3 2 chunks +13 lines, -0 lines 0 comments Download
M cc/test/begin_frame_args_test.cc View 1 2 3 1 chunk +16 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
mithro-old
Hi Brian, This patch makes BeginFrameArgs much nicer to use with gtest. Tim
6 years, 7 months ago (2014-05-08 03:46:17 UTC) #1
brianderson
https://codereview.chromium.org/273823002/diff/1/cc/test/output_test_common.h File cc/test/output_test_common.h (right): https://codereview.chromium.org/273823002/diff/1/cc/test/output_test_common.h#newcode29 cc/test/output_test_common.h:29: bool operator==(BeginFrameArgs lhs, BeginFrameArgs rhs); Is it possible to ...
6 years, 7 months ago (2014-05-08 16:56:32 UTC) #2
mithro-old
PTAL. https://codereview.chromium.org/273823002/diff/1/cc/test/output_test_common.h File cc/test/output_test_common.h (right): https://codereview.chromium.org/273823002/diff/1/cc/test/output_test_common.h#newcode29 cc/test/output_test_common.h:29: bool operator==(BeginFrameArgs lhs, BeginFrameArgs rhs); On 2014/05/08 16:56:32, ...
6 years, 7 months ago (2014-05-08 22:21:21 UTC) #3
brianderson
lgtm if == uses references.
6 years, 7 months ago (2014-05-08 22:23:58 UTC) #4
mithro-old
The CQ bit was checked by mithro@mithis.com
6 years, 7 months ago (2014-05-19 12:32:16 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mithro@mithis.com/273823002/100001
6 years, 7 months ago (2014-05-19 12:32:22 UTC) #6
mithro-old
The CQ bit was checked by mithro@mithis.com
6 years, 7 months ago (2014-05-19 13:15:50 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mithro@mithis.com/273823002/120001
6 years, 7 months ago (2014-05-19 13:16:34 UTC) #8
commit-bot: I haz the power
6 years, 7 months ago (2014-05-19 17:36:37 UTC) #9
Message was sent while issue was closed.
Change committed as 271413

Powered by Google App Engine
This is Rietveld 408576698