Supervised user interstitial: Handle missing custodian profile URL.

Supervised user interstitial: Handle missing custodian profile URL.

Followup to https://codereview.chromium.org/483703002/

Turns out you can have a profile image, but no profile URL. Handle this case by resolving the image URL against a dummy https URL.

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

[Marc Treib, 2014-08-21 09:51:30]

PTAL!

[Bernhard Bauer, 2014-08-21 14:48:46]

LGTM w/ a nit:

https://codereview.chromium.org/495003003/diff/1/chrome/browser/supervised_us...
File chrome/browser/supervised_user/supervised_user_interstitial.cc (right):

https://codereview.chromium.org/495003003/diff/1/chrome/browser/supervised_us...
chrome/browser/supervised_user/supervised_user_interstitial.cc:151: GURL
profile_url = GURL(profile->GetPrefs()->GetString(
Could we rename this to base_url, now that it might not always actually contain
the profile URL?

[Marc Treib, 2014-08-21 15:59:08]

https://codereview.chromium.org/495003003/diff/1/chrome/browser/supervised_us...
File chrome/browser/supervised_user/supervised_user_interstitial.cc (right):

https://codereview.chromium.org/495003003/diff/1/chrome/browser/supervised_us...
chrome/browser/supervised_user/supervised_user_interstitial.cc:151: GURL
profile_url = GURL(profile->GetPrefs()->GetString(
On 2014/08/21 14:48:46, Bernhard Bauer wrote:
> Could we rename this to base_url, now that it might not always actually
contain
> the profile URL?

Done.

[Marc Treib, 2014-08-21 15:59:21]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2014-08-21 16:01:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/treib@chromium.org/495003003/20001

[commit-bot: I haz the power, 2014-08-21 19:20:12]

Committed patchset #2 (20001) as 291153