Remove redundant signals_check_notifier_ schedule.

Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/6182cb1b36dacbcb4cfef9ecb77fe1ee9dbd07b5
Cr-Commit-Position: refs/heads/master@{#434105}

[Yupei Wang, 2016-11-20 08:28:44]

Description was changed from

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.
==========

to

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Yupei Wang, 2016-11-20 08:30:55]

Description was changed from

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Yupei Wang, 2016-11-20 08:30:56]

perryuwang@tencent.com changed reviewers:
+ vmpstr@chromium.org

[Yupei Wang, 2016-11-22 02:03:07]

Hi vmpstr,

I remove redundant signals_check_notifier_ schedule, and add a comment above
signals_.ready_to_activate = need_to_signal_activate
stating that the signals notifier is already scheduled above in the same
function.

Can you review this code, please?

[Yupei Wang, 2016-11-22 03:50:47]

The CQ bit was checked by perryuwang@tencent.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-22 03:51:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 03:51:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-22 03:51:15]

Dry run: No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[vmpstr, 2016-11-22 18:01:53]

lgtm, thanks!

[Yupei Wang, 2016-11-23 01:08:30]

The CQ bit was checked by perryuwang@tencent.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-23 01:09:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-23 03:10:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-23 03:10:03]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Yupei Wang, 2016-11-23 03:11:08]

The CQ bit was checked by perryuwang@tencent.com

[commit-bot: I haz the power, 2016-11-23 03:11:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-23 03:45:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479870668440830, "parent_rev":
"fe8017ac208d8109b3aacfd58df0bd134865db8c", "commit_rev":
"eb4ca6a2011aa3196f40407949aec0ace9fca3cc"}

[commit-bot: I haz the power, 2016-11-23 03:45:26]

Description was changed from

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[commit-bot: I haz the power, 2016-11-23 03:45:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-23 03:47:39]

Description was changed from

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove redundant signals_check_notifier_ schedule.

signals_check_notifier_ is scheduled twice in function
CheckIfMoreTilesNeedToBePrepared. it's redundant and remove it.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/6182cb1b36dacbcb4cfef9ecb77fe1ee9dbd07b5
Cr-Commit-Position: refs/heads/master@{#434105}
==========

[commit-bot: I haz the power, 2016-11-23 03:47:42]

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