Create setup and arrange duplicated the following lines for Issue 443821

Create setup and arrange duplicated the following lines for Issue 443821

  StateMachine state(scheduler_settings);
  state.SetCanStart();
  state.UpdateState(state.NextAction());
  state.CreateAndInitializeOutputSurfaceWithActivatedCommit();
  state.SetVisible(true);
  state.SetCanDraw(true);

BUG=443821
Committed: https://crrev.com/5ed0df28fb122843bf65a1084b0c971c5e083410
Cr-Commit-Position: refs/heads/master@{#313199}

[Jimmy Jo, 2015-01-24 06:45:08]

Patchset #1 (id:1) has been deleted

[Jimmy Jo, 2015-01-24 07:06:36]

Patchset #1 (id:20001) has been deleted

[Jimmy Jo, 2015-01-24 07:15:04]

Patchset #1 (id:40001) has been deleted

[Jimmy Jo, 2015-01-24 07:19:21]

Patchset #1 (id:60001) has been deleted

[Jimmy Jo, 2015-01-24 07:25:00]

jincheol.jo@navercorp.com changed reviewers:
+ brianderson@chromium.org, mithro@mithis.com, simonhong@chromium.org

[Jimmy Jo, 2015-01-24 07:33:24]

Patchset #1 (id:80001) has been deleted

[Jimmy Jo, 2015-01-24 07:36:33]

PTAL!!

[sunnyps, 2015-01-25 03:27:32]

sunnyps@chromium.org changed reviewers:
+ sunnyps@chromium.org

[sunnyps, 2015-01-25 03:27:33]

https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
File cc/scheduler/scheduler_state_machine_unittest.cc (right):

https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine_unittest.cc:52: #define SET_UP_STATE(state,
visible, can_draw)                 \
There's really no need for visible and can_draw to be arguments to this macro
because they are always true.

https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine_unittest.cc:56: state.SetVisible(true);    
                                 \
SetVisible and SetCanDraw must be called with visible and can_draw respectively.

[Jimmy Jo, 2015-01-25 06:05:09]

https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
File cc/scheduler/scheduler_state_machine_unittest.cc (right):

https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine_unittest.cc:52: #define SET_UP_STATE(state,
visible, can_draw)                 \
On 2015/01/25 03:27:33, sunnyps wrote:
> There's really no need for visible and can_draw to be arguments to this macro
> because they are always true. 

Done.

https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine_unittest.cc:56: state.SetVisible(true);    
                                 \
On 2015/01/25 03:27:33, sunnyps wrote:
> SetVisible and SetCanDraw must be called with visible and can_draw
respectively.
Done. I intented to set visible and can_draw but I forgot it.
I corrected my mistake.
Thanks sunnyps.

[sunnyps, 2015-01-26 19:48:45]

On 2015/01/25 06:05:09, Jincheol Jo wrote:
>
https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
> File cc/scheduler/scheduler_state_machine_unittest.cc (right):
> 
>
https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
> cc/scheduler/scheduler_state_machine_unittest.cc:52: #define
SET_UP_STATE(state,
> visible, can_draw)                 \
> On 2015/01/25 03:27:33, sunnyps wrote:
> > There's really no need for visible and can_draw to be arguments to this
macro
> > because they are always true. 
> 
> Done.
> 
>
https://codereview.chromium.org/875613002/diff/100001/cc/scheduler/scheduler_...
> cc/scheduler/scheduler_state_machine_unittest.cc:56: state.SetVisible(true);  
 
>                                  \
> On 2015/01/25 03:27:33, sunnyps wrote:
> > SetVisible and SetCanDraw must be called with visible and can_draw
> respectively.
> Done. I intented to set visible and can_draw but I forgot it.
> I corrected my mistake.
> Thanks sunnyps.

LGTM. You'll need an LGTM from a cc scheduling OWNER too.

[brianderson, 2015-01-27 00:51:32]

lgtm. Thank Jincheol.

[Jimmy Jo, 2015-01-27 00:56:44]

The CQ bit was checked by jincheol.jo@navercorp.com

[Jimmy Jo, 2015-01-27 00:57:52]

On 2015/01/27 00:51:32, brianderson wrote:
> lgtm. Thank Jincheol.

Thank sunny, brian.

[commit-bot: I haz the power, 2015-01-27 00:58:20]

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

[commit-bot: I haz the power, 2015-01-27 01:48:54]

Committed patchset #2 (id:120001)

[commit-bot: I haz the power, 2015-01-27 01:49:40]

Patchset 2 (id:??) landed as
https://crrev.com/5ed0df28fb122843bf65a1084b0c971c5e083410
Cr-Commit-Position: refs/heads/master@{#313199}