Fix some problems with new reauth:

Fix some problems with new reauth:
- use embedded view instead of servicelogin
- don't show settings page when reauth'ing primary account
- use new UMA action for logging reauth

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

[Roger Tawa OOO till Jul 10th, 2014-05-24 03:30:44]

Hi Hui,

Please take a look.  Thanks.

[guohui, 2014-05-26 18:42:05]

lgtm, thanks!

[guohui, 2014-05-26 18:44:43]

https://codereview.chromium.org/300523003/diff/60001/chrome/browser/signin/si...
File chrome/browser/signin/signin_promo.cc (right):

https://codereview.chromium.org/300523003/diff/60001/chrome/browser/signin/si...
chrome/browser/signin/signin_promo.cc:254: switches::IsNewProfileManagement());
nit: could you please add comment here? something like, 

true /* auto_close */, switches::IsNewProfileMnaagement()  /* is_constrained */

[Roger Tawa OOO till Jul 10th, 2014-05-27 16:32:16]

The CQ bit was checked by rogerta@chromium.org

[Roger Tawa OOO till Jul 10th, 2014-05-27 16:32:39]

Thanks Hui.

https://codereview.chromium.org/300523003/diff/60001/chrome/browser/signin/si...
File chrome/browser/signin/signin_promo.cc (right):

https://codereview.chromium.org/300523003/diff/60001/chrome/browser/signin/si...
chrome/browser/signin/signin_promo.cc:254: switches::IsNewProfileManagement());
On 2014/05/26 18:44:43, guohui wrote:
> nit: could you please add comment here? something like, 
> 
> true /* auto_close */, switches::IsNewProfileMnaagement()  /* is_constrained
*/

Done.

[commit-bot: I haz the power, 2014-05-27 16:32:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rogerta@chromium.org/300523003/100001

[commit-bot: I haz the power, 2014-05-27 18:12:14]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[commit-bot: I haz the power, 2014-05-27 18:15:14]

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

[commit-bot: I haz the power, 2014-05-27 18:15:14]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)

[Roger Tawa OOO till Jul 10th, 2014-05-27 20:54:13]

The CQ bit was checked by rogerta@chromium.org

[commit-bot: I haz the power, 2014-05-27 20:56:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rogerta@chromium.org/300523003/100001

[commit-bot: I haz the power, 2014-05-27 22:27:53]

Change committed as 273054