Upon deletion of the profile don't fallback on current if it supervised.

Upon deletion of the profile don't fallback on current if it supervised.

BUG=686035
R=anthonyvd@chromium.org, bauerb@chromium.org

Review-Url: https://codereview.chromium.org/2659803003
Cr-Commit-Position: refs/heads/master@{#447235}
Committed: https://chromium.googlesource.com/chromium/src/+/c5cdb47188549b4d5c9ce8905dc8d1edd797cd07

[Bernhard Bauer, 2017-01-27 16:17:40]

https://codereview.chromium.org/2659803003/diff/1/chrome/browser/profiles/pro...
File chrome/browser/profiles/profile_manager.cc (right):

https://codereview.chromium.org/2659803003/diff/1/chrome/browser/profiles/pro...
chrome/browser/profiles/profile_manager.cc:1377: last_used_profile != nullptr &&
Can the last used profile actually be null?

[palar, 2017-01-30 10:51:18]

On 2017/01/27 16:17:40, Bernhard Bauer wrote:
>
https://codereview.chromium.org/2659803003/diff/1/chrome/browser/profiles/pro...
> File chrome/browser/profiles/profile_manager.cc (right):
> 
>
https://codereview.chromium.org/2659803003/diff/1/chrome/browser/profiles/pro...
> chrome/browser/profiles/profile_manager.cc:1377: last_used_profile != nullptr
&&
> Can the last used profile actually be null?

It shouldn't, but if prefs::kProfileLastUsed will be set to some inactive
profile path it will.

[Bernhard Bauer, 2017-01-30 15:20:11]

LGTM

[palar, 2017-01-30 15:27:56]

The CQ bit was checked by palar@yandex-team.ru

[commit-bot: I haz the power, 2017-01-30 15:28:18]

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, 2017-01-30 15:37:26]

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

[commit-bot: I haz the power, 2017-01-30 15:37:27]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[anthonyvd, 2017-01-31 14:43:57]

lgtm

[palar, 2017-01-31 15:22:58]

The CQ bit was checked by palar@yandex-team.ru

[commit-bot: I haz the power, 2017-01-31 15:23:10]

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, 2017-01-31 15:26:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485876178079220, "parent_rev":
"85ea3b61e9924a99152b23249143c815860e9fd6", "commit_rev":
"c5cdb47188549b4d5c9ce8905dc8d1edd797cd07"}

[commit-bot: I haz the power, 2017-01-31 15:27:12]

Description was changed from

==========
Upon deletion of the profile don't fallback on current if it supervised.

BUG=686035

R=anthonyvd@chromium.org, bauerb@chromium.org
==========

to

==========
Upon deletion of the profile don't fallback on current if it supervised.

BUG=686035

R=anthonyvd@chromium.org, bauerb@chromium.org

Review-Url: https://codereview.chromium.org/2659803003
Cr-Commit-Position: refs/heads/master@{#447235}
Committed:
https://chromium.googlesource.com/chromium/src/+/c5cdb47188549b4d5c9ce8905dc8...
==========

[commit-bot: I haz the power, 2017-01-31 15:27:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c5cdb47188549b4d5c9ce8905dc8...