Remove logging added for debugging the lockscreen crashes.

Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/23371b0a9c7cd223174f4e4580823ead12036f03
Cr-Commit-Position: refs/heads/master@{#400817}

[jdufault, 2016-06-15 18:06:19]

Description was changed from

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
==========

to

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[jdufault, 2016-06-15 23:13:02]

Patchset #1 (id:1) has been deleted

[jdufault, 2016-06-16 18:04:19]

Description was changed from

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[jdufault, 2016-06-16 18:04:19]

jdufault@chromium.org changed reviewers:
+ oshima@chromium.org, xiyuan@chromium.org

[jdufault, 2016-06-16 18:05:28]

Oshima, PTAL at ash/
Xiyuan, PTAL at chrome/browser/.

This CL just removes some logging used for debugging crbug.com/452599.

[Daniel Erat, 2016-06-16 19:54:08]

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

[Daniel Erat, 2016-06-16 19:54:09]

https://codereview.chromium.org/2070053003/diff/20001/ash/wm/lock_state_contr...
File ash/wm/lock_state_controller.cc (right):

https://codereview.chromium.org/2070053003/diff/20001/ash/wm/lock_state_contr...
ash/wm/lock_state_controller.cc:280: << "(lock_fail_timer_.IsRunning: " <<
lock_fail_timer_.IsRunning()
i don't understand the value of logging lock_fail_timer_'s state here.
OnLockFailTimeout() is only called by lock_fail_timer_, and it's a
base::OneShotTimer(). is it even possible for it to be running while in this
method? it doesn't seem like it.

[jdufault, 2016-06-20 16:36:01]

https://codereview.chromium.org/2070053003/diff/20001/ash/wm/lock_state_contr...
File ash/wm/lock_state_controller.cc (right):

https://codereview.chromium.org/2070053003/diff/20001/ash/wm/lock_state_contr...
ash/wm/lock_state_controller.cc:280: << "(lock_fail_timer_.IsRunning: " <<
lock_fail_timer_.IsRunning()
On 2016/06/16 19:54:08, Daniel Erat wrote:
> i don't understand the value of logging lock_fail_timer_'s state here.
> OnLockFailTimeout() is only called by lock_fail_timer_, and it's a
> base::OneShotTimer(). is it even possible for it to be running while in this
> method? it doesn't seem like it.

Done.

[Daniel Erat, 2016-06-20 16:39:10]

lgtm

[xiyuan, 2016-06-20 16:41:07]

lgtm

[oshima, 2016-06-20 21:36:46]

lgtm

[jdufault, 2016-06-20 21:44:46]

The CQ bit was checked by jdufault@chromium.org

[commit-bot: I haz the power, 2016-06-20 21:45:12]

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

[commit-bot: I haz the power, 2016-06-20 22:50:54]

Description was changed from

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-06-20 22:50:56]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-20 22:55:39]

Description was changed from

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Remove logging added for debugging the lockscreen crashes.

BUG=452599
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/23371b0a9c7cd223174f4e4580823ead12036f03
Cr-Commit-Position: refs/heads/master@{#400817}
==========

[commit-bot: I haz the power, 2016-06-20 22:55:40]

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