Fixed a few places that incorrectly assumed that profile was initialized

Fixed a few places that incorrectly assumed that profile was initialized

While testing a fix for crbug.com/532317, I found a few places in the code
that assumed that logged-in users would always have an associated profile.

However, it can take arbitrarily long from the time that a user logs in until
their profile finishes initialization, so code should not make this assumption.

This CL fixes the extension garbage collector and ProfileHelper to properly
handle uninitialized profiles.

BUG=592993
Committed: https://crrev.com/f17bcd66f045d27078a29fc0ace0fa993cec3274
Cr-Commit-Position: refs/heads/master@{#383960}

[Andrew T Wilson (Slow), 2016-03-08 12:35:49]

Description was changed from

==========
Fixed a few places that incorrectly assumed that profile was initialized

While testing a fix for crbug.com/532317, I found a few places in the code
that assumed that logged-in users would always have an associated profile.

However, it can take arbitrarily long from the time that a user logs in until
their profile finishes initialization, so code should not make this assumption.

This CL fixes the extension garbage collector and ProfileHelper to properly
handle uninitialized profiles.

BUG=532317
==========

to

==========
Fixed a few places that incorrectly assumed that profile was initialized

While testing a fix for crbug.com/532317, I found a few places in the code
that assumed that logged-in users would always have an associated profile.

However, it can take arbitrarily long from the time that a user logs in until
their profile finishes initialization, so code should not make this assumption.

This CL fixes the extension garbage collector and ProfileHelper to properly
handle uninitialized profiles.

BUG=592993
==========

[Andrew T Wilson (Slow), 2016-03-08 16:32:07]

atwilson@chromium.org changed reviewers:
+ finnur@chromium.org, skuhne@chromium.org

[Andrew T Wilson (Slow), 2016-03-08 16:32:07]

skuhne: PTAL at profile_manager.cc
finnur: PTAL at extensions/*

[Finnur, 2016-03-09 13:49:30]

extensions LGTM

[Mr4D (OOO till 08-26), 2016-03-23 21:46:49]

lgtm

[Andrew T Wilson (Slow), 2016-03-30 12:20:31]

The CQ bit was checked by atwilson@chromium.org

[commit-bot: I haz the power, 2016-03-30 12:20:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1773113002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1773113002/20001

[commit-bot: I haz the power, 2016-03-30 13:08:20]

Description was changed from

==========
Fixed a few places that incorrectly assumed that profile was initialized

While testing a fix for crbug.com/532317, I found a few places in the code
that assumed that logged-in users would always have an associated profile.

However, it can take arbitrarily long from the time that a user logs in until
their profile finishes initialization, so code should not make this assumption.

This CL fixes the extension garbage collector and ProfileHelper to properly
handle uninitialized profiles.

BUG=592993
==========

to

==========
Fixed a few places that incorrectly assumed that profile was initialized

While testing a fix for crbug.com/532317, I found a few places in the code
that assumed that logged-in users would always have an associated profile.

However, it can take arbitrarily long from the time that a user logs in until
their profile finishes initialization, so code should not make this assumption.

This CL fixes the extension garbage collector and ProfileHelper to properly
handle uninitialized profiles.

BUG=592993
==========

[commit-bot: I haz the power, 2016-03-30 13:08:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-30 13:09:56]

Description was changed from

==========
Fixed a few places that incorrectly assumed that profile was initialized

While testing a fix for crbug.com/532317, I found a few places in the code
that assumed that logged-in users would always have an associated profile.

However, it can take arbitrarily long from the time that a user logs in until
their profile finishes initialization, so code should not make this assumption.

This CL fixes the extension garbage collector and ProfileHelper to properly
handle uninitialized profiles.

BUG=592993
==========

to

==========
Fixed a few places that incorrectly assumed that profile was initialized

While testing a fix for crbug.com/532317, I found a few places in the code
that assumed that logged-in users would always have an associated profile.

However, it can take arbitrarily long from the time that a user logs in until
their profile finishes initialization, so code should not make this assumption.

This CL fixes the extension garbage collector and ProfileHelper to properly
handle uninitialized profiles.

BUG=592993

Committed: https://crrev.com/f17bcd66f045d27078a29fc0ace0fa993cec3274
Cr-Commit-Position: refs/heads/master@{#383960}
==========

[commit-bot: I haz the power, 2016-03-30 13:09:57]

Patchset 2 (id:??) landed as
https://crrev.com/f17bcd66f045d27078a29fc0ace0fa993cec3274
Cr-Commit-Position: refs/heads/master@{#383960}