Make EXPECT_ACTION_UPDATE_STATE failure output easier to understand.

Make EXPECT_ACTION_UPDATE_STATE failure output easier to understand.

Changes;
-------------------------------------------------------------------------------
Value of: state.NextAction()
  Actual: 1
Expected: SchedulerStateMachine::ACTION_DRAW_AND_SWAP_ABORT
Which is: 8
-------------------------------------------------------------------------------

to

-------------------------------------------------------------------------------
Value of: SchedulerStateMachine::ActionToString(state.NextAction())
  Actual: "ACTION_NONE"
Expected: SchedulerStateMachine::ActionToString( SchedulerStateMachine::ACTION_DRAW_AND_SWAP_ABORT)
Which is: "ACTION_DRAW_AND_SWAP_ABORT"
-------------------------------------------------------------------------------

BUG=416749
Committed: https://crrev.com/06968cb7a6534e631481d0f20b3c666cc2f98805
Cr-Commit-Position: refs/heads/master@{#299282}

[mithro-old, 2014-10-13 04:08:05]

mithro@mithis.com changed reviewers:
+ skyostil@chromium.org, sunnyps@chromium.org

[mithro-old, 2014-10-13 04:08:06]

Hi,

A small change split out of https://codereview.chromium.org/621883002/ which I
don't think will be landable.

Tim 'mithro' Ansell

===============================================================================
Make EXPECT_ACTION_UPDATE_STATE failure output easier to understand.

Changes;
-------------------------------------------------------------------------------
Value of: state.NextAction()
  Actual: 1
Expected: SchedulerStateMachine::ACTION_DRAW_AND_SWAP_ABORT
Which is: 8
-------------------------------------------------------------------------------

to

-------------------------------------------------------------------------------
Value of: SchedulerStateMachine::ActionToString(state.NextAction())
  Actual: "ACTION_NONE"
Expected:
SchedulerStateMachine::ActionToString(SchedulerStateMachine::ACTION_DRAW_AND_SWAP_ABORT)
Which is: "ACTION_DRAW_AND_SWAP_ABORT"
-------------------------------------------------------------------------------

BUG=416749

[Sami, 2014-10-13 09:43:05]

lgtm modulo one question.

https://codereview.chromium.org/639963006/diff/1/cc/scheduler/scheduler_state...
File cc/scheduler/scheduler_state_machine_unittest.cc (right):

https://codereview.chromium.org/639963006/diff/1/cc/scheduler/scheduler_state...
cc/scheduler/scheduler_state_machine_unittest.cc:13:
EXPECT_STREQ(SchedulerStateMachine::ActionToString(action),                \
I'm guessing this doesn't hugely affect the test's run time?

[mithro-old, 2014-10-13 10:29:10]

https://codereview.chromium.org/639963006/diff/1/cc/scheduler/scheduler_state...
File cc/scheduler/scheduler_state_machine_unittest.cc (right):

https://codereview.chromium.org/639963006/diff/1/cc/scheduler/scheduler_state...
cc/scheduler/scheduler_state_machine_unittest.cc:13:
EXPECT_STREQ(SchedulerStateMachine::ActionToString(action),                \
On 2014/10/13 09:43:05, Sami wrote:
> I'm guessing this doesn't hugely affect the test's run time?

The change is well within noise on a z620.

[mithro-old, 2014-10-13 10:29:16]

The CQ bit was checked by mithro@mithis.com

[commit-bot: I haz the power, 2014-10-13 10:29:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/639963006/1

[commit-bot: I haz the power, 2014-10-13 10:57:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-13 10:58:01]

Patchset 1 (id:??) landed as
https://crrev.com/06968cb7a6534e631481d0f20b3c666cc2f98805
Cr-Commit-Position: refs/heads/master@{#299282}