Fix background page loading

Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  

I was able to get rid of the popup that would happen
sometimes by being a bit more patient on the auth
library to load its thing.

The main culprit was the fact that we were waiting until
the auth-signin's attached phase to do anything, which
apparently requires tab focus.  The ready phase can fire
while the tab is still hidden, so we use that instead.

I also made the token "offline", seems to have helped with
more consistent credential management by the 
google-signin-aware element.  It may have been unnecessary,
but it works nonetheless.

BUG=653703
Committed: https://github.com/luci/luci-py/commit/39e913eeed7f17dc3147c588ec7cb85125742056

[kjlubick, 2016-10-07 17:00:52]

Description was changed from

==========
Fix background page loading

BUG=653703
==========

to

==========
Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  You will need to
disable popups to allow the "auto signin" to work.

BUG=653703
==========

[kjlubick, 2016-10-07 17:54:50]

Description was changed from

==========
Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  You will need to
disable popups to allow the "auto signin" to work.

BUG=653703
==========

to

==========
Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  

I was able to get rid of the popup that would happen
sometimes by being a bit more patient on the auth
library to load its thing.

The main culprit was the fact that we were waiting until
the auth-signin's attached phase to do anything, which
apparently requires tab focus.  The ready phase can fire
while the tab is still hidden, so we use that instead.

BUG=653703
==========

[kjlubick, 2016-10-07 17:56:12]

Description was changed from

==========
Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  

I was able to get rid of the popup that would happen
sometimes by being a bit more patient on the auth
library to load its thing.

The main culprit was the fact that we were waiting until
the auth-signin's attached phase to do anything, which
apparently requires tab focus.  The ready phase can fire
while the tab is still hidden, so we use that instead.

BUG=653703
==========

to

==========
Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  

I was able to get rid of the popup that would happen
sometimes by being a bit more patient on the auth
library to load its thing.

The main culprit was the fact that we were waiting until
the auth-signin's attached phase to do anything, which
apparently requires tab focus.  The ready phase can fire
while the tab is still hidden, so we use that instead.

I also made the token "offline", seems to have helped with
more consistent credential management by the 
google-signin-aware element.  It may have been unnecessary,
but it works nonetheless.

BUG=653703
==========

[kjlubick, 2016-10-07 17:57:00]

kjlubick@google.com changed reviewers:
+ jcgregorio@google.com, stephana@google.com

[kjlubick, 2016-10-07 17:57:02]

Try it out at
https://2389-44bbd1f-tainted-kjlubick-dot-chromium-swarm-dev.appspot.com/newui/

[jcgregorio, 2016-10-07 17:59:09]

lgtm

[KevinL, 2016-10-07 17:59:23]

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

[KevinL, 2016-10-07 17:59:23]

lgtm

[KevinL, 2016-10-07 17:59:44]

The CQ bit was checked by kjlubick@chromium.org

[commit-bot: I haz the power, 2016-10-07 17:59:47]

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-10-07 18:03:18]

Description was changed from

==========
Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  

I was able to get rid of the popup that would happen
sometimes by being a bit more patient on the auth
library to load its thing.

The main culprit was the fact that we were waiting until
the auth-signin's attached phase to do anything, which
apparently requires tab focus.  The ready phase can fire
while the tab is still hidden, so we use that instead.

I also made the token "offline", seems to have helped with
more consistent credential management by the 
google-signin-aware element.  It may have been unnecessary,
but it works nonetheless.

BUG=653703
==========

to

==========
Fix background page loading

You should now be able to open pages in a new tab
and they will start loading.  

I was able to get rid of the popup that would happen
sometimes by being a bit more patient on the auth
library to load its thing.

The main culprit was the fact that we were waiting until
the auth-signin's attached phase to do anything, which
apparently requires tab focus.  The ready phase can fire
while the tab is still hidden, so we use that instead.

I also made the token "offline", seems to have helped with
more consistent credential management by the 
google-signin-aware element.  It may have been unnecessary,
but it works nonetheless.

BUG=653703

Committed:
https://github.com/luci/luci-py/commit/39e913eeed7f17dc3147c588ec7cb85125742056
==========

[commit-bot: I haz the power, 2016-10-07 18:03:18]

Committed patchset #2 (id:20001) as
https://github.com/luci/luci-py/commit/39e913eeed7f17dc3147c588ec7cb85125742056