[Findit] Redirect to original page after login for triage.

[Findit] Redirect to original page after login for triage.

BUG=665207
R=chanli, lijeffrey

Committed: https://chromium.googlesource.com/infra/infra/+/c1f333be8ca8b0f43332ca777ebcc7d148745673

[stgao, 2016-11-30 06:31:56]

ptal

[chanli, 2016-11-30 07:19:00]

On 2016/11/30 06:31:56, stgao (slow on Monday) wrote:
> ptal

lgtm with a nit

[chanli, 2016-11-30 07:19:06]

https://codereview.chromium.org/2537143003/diff/1/appengine/findit/common/bas...
File appengine/findit/common/base_handler.py (right):

https://codereview.chromium.org/2537143003/diff/1/appengine/findit/common/bas...
appengine/findit/common/base_handler.py:164: # file, etc. Thus return to the
original page if available.
Nit: This comment needs to be updated.

[stgao, 2016-11-30 17:52:36]

Patchset #2 (id:20001) has been deleted

[stgao, 2016-11-30 17:53:09]

https://codereview.chromium.org/2537143003/diff/1/appengine/findit/common/bas...
File appengine/findit/common/base_handler.py (right):

https://codereview.chromium.org/2537143003/diff/1/appengine/findit/common/bas...
appengine/findit/common/base_handler.py:164: # file, etc. Thus return to the
original page if available.
On 2016/11/30 07:19:06, chanli wrote:
> Nit: This comment needs to be updated.

Done.

[stgao, 2016-11-30 17:53:14]

The CQ bit was checked by stgao@chromium.org

[stgao, 2016-11-30 17:53:14]

The patchset sent to the CQ was uploaded after l-g-t-m from chanli@chromium.org
Link to the patchset: https://codereview.chromium.org/2537143003/#ps40001
(title: ".")

[commit-bot: I haz the power, 2016-11-30 17:53:20]

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

[lijeffrey, 2016-11-30 18:05:53]

lgtm

[commit-bot: I haz the power, 2016-11-30 18:06:45]

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

[commit-bot: I haz the power, 2016-11-30 18:06:45]

Try jobs failed on following builders:
  Infra Linux Trusty 64 Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/32cf36568eb28b10)

[stgao, 2016-11-30 19:34:26]

The CQ bit was checked by stgao@chromium.org

[stgao, 2016-11-30 19:34:27]

The patchset sent to the CQ was uploaded after l-g-t-m from chanli@chromium.org,
lijeffrey@chromium.org
Link to the patchset: https://codereview.chromium.org/2537143003/#ps60001
(title: "Fix test failures & rename var.")

[commit-bot: I haz the power, 2016-11-30 19:34:33]

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-30 19:49:24]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1480534466838500,
"parent_rev": "a51f9954ce1af77d0e0c5468642f0d399936688f", "commit_rev":
"c1f333be8ca8b0f43332ca777ebcc7d148745673"}

[commit-bot: I haz the power, 2016-11-30 19:49:27]

Description was changed from

==========
[Findit] Redirect to original page after login for triage.

BUG=665207
R=chanli, lijeffrey
==========

to

==========
[Findit] Redirect to original page after login for triage.

BUG=665207
R=chanli, lijeffrey

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

[commit-bot: I haz the power, 2016-11-30 19:49:28]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/infra/infra/+/c1f333be8ca8b0f43332ca777ebcc...

[stgao, 2017-03-17 18:58:50]

Description was changed from

==========
[Findit] Redirect to original page after login for triage.

BUG=665207
R=chanli, lijeffrey

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

to

==========
[Findit] Redirect to original page after login for triage.

BUG=665207
R=chanli, lijeffrey

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

[stgao, 2017-03-17 18:59:26]

A revert of this CL (patchset #3 id:60001) has been created in
https://d1a128a62-dot-chromiumcodereview-hr.appspot.com/2759743002/ by
stgao@chromium.org.

The reason for reverting is: testing, don't commit.

[findit-for-me, 2017-03-17 19:00:07]

A revert of this CL (patchset #3 id:60001) has been created in
https://d1a128a62-dot-chromiumcodereview-hr.appspot.com/2748183007/ by
findit-for-me@appspot.gserviceaccount.com.

The reason for reverting is: Testing revert from Findit app, please ignore
this..

[findit-for-me, 2017-03-17 19:13:25]

A revert of this CL (patchset #3 id:60001) has been created in
https://codereview.chromium.org/2756973002/ by
findit-for-me@appspot.gserviceaccount.com.

The reason for reverting is: Testing revert from Findit app, please ignore
this..