Fix the bug which is login dialog won't close after the second sign-in to chrome with different acc…

Fix the bug which is login dialog won't close after the second sign-in to chrome with different account on the same profile.

This is the follow up of CLs:
1) https://chromium.googlesource.com/chromium/src/+/ff9fc46a4fdfb7c2975c867d8459e3cebe2e32fb
and
2) https://chromium.googlesource.com/chromium/src/+/5db87096a32b73bd2aaca04710152de495ba50ea

The javascript message that close sign in dialog will be sent regardless before. However, after the refactor in the second CL above, it's only sent once the profile_path is not empty. It didn't cause any issue because the JS message will be sent again in the OnClientOAuthSuccess()

However, the OnClientOAuthSuccess' JS message will only be sent while force sign in is enabled after the first CL landed which is used to solve crbug.com/667227. It means that if the profile_path is empty (second sign in with different account in the same profile), the sign in dialog will never be closed.

Moving the profile_path empty check into UnlockProfileAndHideLoginUI() so that the javascript message will be sent no matter the profile_path is empty or not.

BUG=668619
Review-Url: https://codereview.chromium.org/2528183002
Cr-Commit-Position: refs/heads/master@{#434668}
(cherry picked from commit 0233a4c8e8eaa98fd46de68f975286b438751211)

Committed: https://chromium.googlesource.com/chromium/src/+/6500dfd08d8152e3d4ea0207d17b0111291489f1

[Patrick Monette, 2016-11-30 23:58:19]

Description was changed from

==========
Fix the bug which is login dialog won't close after the second sign-in to chrome
with different account on the same profile.

This is the follow up of CLs:
1)
https://chromium.googlesource.com/chromium/src/+/ff9fc46a4fdfb7c2975c867d8459...
and
2)
https://chromium.googlesource.com/chromium/src/+/5db87096a32b73bd2aaca0471015...

The javascript message that close sign in dialog will be sent regardless before.
However, after the refactor in the second CL above, it's only sent once the
profile_path is not empty. It didn't cause any issue because the JS message will
be sent again in the OnClientOAuthSuccess()

However, the OnClientOAuthSuccess' JS message will only be sent while force sign
in is enabled after the first CL landed which is used to solve crbug.com/667227.
It means that if the profile_path is empty (second sign in with different
account in the same profile), the sign in dialog will never be closed.

Moving the profile_path empty check into UnlockProfileAndHideLoginUI() so that
the javascript message will be sent no matter the profile_path is empty or not.

BUG=668619

Review-Url: https://codereview.chromium.org/2528183002
Cr-Commit-Position: refs/heads/master@{#434668}
(cherry picked from commit 0233a4c8e8eaa98fd46de68f975286b438751211)
==========

to

==========
Fix the bug which is login dialog won't close after the second sign-in to chrome
with different account on the same profile.

This is the follow up of CLs:
1)
https://chromium.googlesource.com/chromium/src/+/ff9fc46a4fdfb7c2975c867d8459...
and
2)
https://chromium.googlesource.com/chromium/src/+/5db87096a32b73bd2aaca0471015...

The javascript message that close sign in dialog will be sent regardless before.
However, after the refactor in the second CL above, it's only sent once the
profile_path is not empty. It didn't cause any issue because the JS message will
be sent again in the OnClientOAuthSuccess()

However, the OnClientOAuthSuccess' JS message will only be sent while force sign
in is enabled after the first CL landed which is used to solve crbug.com/667227.
It means that if the profile_path is empty (second sign in with different
account in the same profile), the sign in dialog will never be closed.

Moving the profile_path empty check into UnlockProfileAndHideLoginUI() so that
the javascript message will be sent no matter the profile_path is empty or not.

BUG=668619

Review-Url: https://codereview.chromium.org/2528183002
Cr-Commit-Position: refs/heads/master@{#434668}
(cherry picked from commit 0233a4c8e8eaa98fd46de68f975286b438751211)

Committed:
https://chromium.googlesource.com/chromium/src/+/6500dfd08d8152e3d4ea0207d17b...
==========

[Patrick Monette, 2016-11-30 23:58:21]

Committed patchset #1 (id:1) manually as
6500dfd08d8152e3d4ea0207d17b0111291489f1.