Add spinner to the 'managed user import' overlay

Add spinner to the 'managed user import' overlay

BUG=292418
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223404

[ibra, 2013-09-12 13:21:19]

Hey Sergiu, please review, thanks!

[Sergiu, 2013-09-16 08:46:04]

lgtm

https://codereview.chromium.org/23533047/diff/4001/chrome/browser/resources/o...
File chrome/browser/resources/options/managed_user_import.js (right):

https://codereview.chromium.org/23533047/diff/4001/chrome/browser/resources/o...
chrome/browser/resources/options/managed_user_import.js:39: var self = this;
I'd move this just before the onclick handler, it's the only place you're using
it so it is clearer that they're connected.

[ibra, 2013-09-16 09:04:14]

+ Bernhard, please take a look. Thanks!

https://codereview.chromium.org/23533047/diff/4001/chrome/browser/resources/o...
File chrome/browser/resources/options/managed_user_import.js (right):

https://codereview.chromium.org/23533047/diff/4001/chrome/browser/resources/o...
chrome/browser/resources/options/managed_user_import.js:39: var self = this;
On 2013/09/16 08:46:04, Sergiu wrote:
> I'd move this just before the onclick handler, it's the only place you're
using
> it so it is clearer that they're connected.

Done.

[Sergiu, 2013-09-16 09:07:13]

On 2013/09/16 09:04:14, ibra wrote:
> + Bernhard, please take a look. Thanks!
> 
>
https://codereview.chromium.org/23533047/diff/4001/chrome/browser/resources/o...
> File chrome/browser/resources/options/managed_user_import.js (right):
> 
>
https://codereview.chromium.org/23533047/diff/4001/chrome/browser/resources/o...
> chrome/browser/resources/options/managed_user_import.js:39: var self = this;
> On 2013/09/16 08:46:04, Sergiu wrote:
> > I'd move this just before the onclick handler, it's the only place you're
> using
> > it so it is clearer that they're connected.
> 
> Done.

Also please add a bug number to this one as well.

[Bernhard Bauer, 2013-09-16 09:12:22]

lgtm

[commit-bot: I haz the power, 2013-09-16 17:37:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ibraaaa@chromium.org/23533047/9001

[commit-bot: I haz the power, 2013-09-16 17:46:06]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[commit-bot: I haz the power, 2013-09-16 17:54:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ibraaaa@chromium.org/23533047/9001

[commit-bot: I haz the power, 2013-09-16 20:41:08]

Change committed as 223404