Replace ProfileIOData::http_server_properties_manager_ with http_server_properties_.

Replace ProfileIOData::http_server_properties_manager_ with http_server_properties_.

ProfileImplIOData::http_server_properties_manager_ is kept in sync and used to access the implementation.

This change moves ownership of HttpServerPropertiesImpl from OffTheRecordProfileIOData to ProfileIOData and
ensures http_server_properties_ outlives ChromeURLRequestContexts owned by ProfileIOData.

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

[willchan no longer on Chromium, 2012-12-03 23:51:07]

lgtm

https://codereview.chromium.org/11316320/diff/1/chrome/browser/profiles/off_t...
File chrome/browser/profiles/off_the_record_profile_io_data.h (right):

https://codereview.chromium.org/11316320/diff/1/chrome/browser/profiles/off_t...
chrome/browser/profiles/off_the_record_profile_io_data.h:21: class
HttpServerPropertiesImpl;
Goodbye

https://codereview.chromium.org/11316320/diff/1/chrome/browser/profiles/profi...
File chrome/browser/profiles/profile_impl_io_data.h (right):

https://codereview.chromium.org/11316320/diff/1/chrome/browser/profiles/profi...
chrome/browser/profiles/profile_impl_io_data.h:195: // Same as
|ProfileIOData::http_server_properties_|, owned there.
Explain it's owned there to maintain correct destruction ordering.

[commit-bot: I haz the power, 2012-12-04 16:13:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/szym@chromium.org/11316320/8001

[commit-bot: I haz the power, 2012-12-04 18:59:04]

Change committed as 171012