chromium_build_stats: redirect login url if not logged in.

chromium_build_stats: redirect login url if not logged in.

BUG=606647, 619718
Committed: https://chromium.googlesource.com/infra/infra/+/dc363e8dd39d08738324fb1f43caba38b6606317

[ukai, 2016-07-05 04:52:42]

Description was changed from

==========
chromium_build_stats: redirect login url if not logged in.

BUG=606647,619718
==========

to

==========
chromium_build_stats: redirect login url if not logged in.

BUG=606647,619718
==========

[ukai, 2016-07-05 04:52:42]

ukai@chromium.org changed reviewers:
+ shinyak@chromium.org, tikuta@chromium.org, yyanagisawa@chromium.org

[Yoshisato Yanagisawa, 2016-07-05 05:00:05]

lgtm w/ comment

https://codereview.chromium.org/2125513002/diff/1/appengine/chromium_build_st...
File appengine/chromium_build_stats/default/compiler_proxy_log.go (right):

https://codereview.chromium.org/2125513002/diff/1/appengine/chromium_build_st...
appengine/chromium_build_stats/default/compiler_proxy_log.go:120: if
!strings.HasSuffix(u.Email, "@google.com") {
maybe intended but L117 and L120 are the same?

[shinyak (Google), 2016-07-05 05:09:36]

shinyak@google.com changed reviewers:
+ shinyak@google.com

[shinyak (Google), 2016-07-05 05:09:36]

lgtm

[tikuta, 2016-07-05 06:07:14]

lgtm

[ukai, 2016-07-06 07:55:36]

The CQ bit was checked by ukai@chromium.org

[commit-bot: I haz the power, 2016-07-06 07:55:48]

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-07-06 08:12:01]

Description was changed from

==========
chromium_build_stats: redirect login url if not logged in.

BUG=606647,619718
==========

to

==========
chromium_build_stats: redirect login url if not logged in.

BUG=606647,619718

Committed:
https://chromium.googlesource.com/infra/infra/+/dc363e8dd39d08738324fb1f43cab...
==========

[commit-bot: I haz the power, 2016-07-06 08:12:02]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/infra/infra/+/dc363e8dd39d08738324fb1f43cab...