Removal of Profile from content part 6.

Removal of Profile from content part 6.

The concept of some WebUIs not being based on TabContents lived for two weeks (added r62478, removed r63713) but this support code was never removed.

BUG=76788
TEST=no change visible

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96390

[jam, 2011-08-09 16:11:19]

lgtm

what do you think of having a convenience function like Profile::FromWebUI?

http://codereview.chromium.org/7554008/diff/8001/chrome/browser/ui/webui/hist...
File chrome/browser/ui/webui/history_ui.cc (right):

http://codereview.chromium.org/7554008/diff/8001/chrome/browser/ui/webui/hist...
chrome/browser/ui/webui/history_ui.cc:177:
Profile::FromBrowserContext(web_ui_->tab_contents()->browser_context());
nit: given how many times I see this line so far, do you think it would make
sense to have a convenience function like Profile::FromWebUI?

[Avi (use Gerrit), 2011-08-09 16:17:17]

Feels weird to me a bit, that Profile would know about WebUI explicitly. You
don't think so? Let me ask around.

[jam, 2011-08-09 20:22:57]

On 2011/08/09 16:17:17, Avi wrote:
> Feels weird to me a bit, that Profile would know about WebUI explicitly. You
> don't think so? Let me ask around.

yeah I felt the same way, that's why I wanted to check with you. i don't feel
strongly.

[Avi (use Gerrit), 2011-08-09 20:25:31]

Miranda's OK with it, so I made the change. I'm doing a build but will have it
up soon.

[Avi (use Gerrit), 2011-08-09 22:09:06]

done; ptal

[jam, 2011-08-10 02:50:21]

lgtm

[Avi (use Gerrit), 2011-08-10 17:44:54]

Miranda, need a profile owner.

[Miranda Callahan, 2011-08-10 19:17:52]

On 2011/08/10 17:44:54, Avi wrote:
> Miranda, need a profile owner.

LGTM