Add support for content::Referrer to prerender

Add support for content::Referrer to prerender

BUG=105028
TEST=none


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

[jochen (gone - plz use gerrit), 2011-12-05 16:53:45]

Chris, please review

John: content/ approval

[jam, 2011-12-05 17:45:09]

content lgtm

[cbentzel, 2011-12-05 18:03:29]

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
File chrome/browser/prerender/prerender_browsertest.cc (right):

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
chrome/browser/prerender/prerender_browsertest.cc:205: const content::Referrer&
referrer) {
Should have OVERRIDE - I can take care of this in a separate CL.

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
File chrome/browser/prerender/prerender_manager.cc (right):

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
chrome/browser/prerender/prerender_manager.cc:776: const content::Refeerrer&
referrer,
BUG: Referrer

http://codereview.chromium.org/8801012/diff/1/chrome/browser/renderer_host/ch...
File chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
(right):

http://codereview.chromium.org/8801012/diff/1/chrome/browser/renderer_host/ch...
chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc:36:
#include "content/public/common/referrer.h"
Why does this need to be included? Looks like forward declaration is sufficient.

[jochen (gone - plz use gerrit), 2011-12-05 20:46:34]

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
File chrome/browser/prerender/prerender_browsertest.cc (right):

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
chrome/browser/prerender/prerender_browsertest.cc:205: const content::Referrer&
referrer) {
On 2011/12/05 18:03:29, cbentzel wrote:
> Should have OVERRIDE - I can take care of this in a separate CL.

Done.

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
File chrome/browser/prerender/prerender_manager.cc (right):

http://codereview.chromium.org/8801012/diff/1/chrome/browser/prerender/preren...
chrome/browser/prerender/prerender_manager.cc:776: const content::Refeerrer&
referrer,
On 2011/12/05 18:03:29, cbentzel wrote:
> BUG: Referrer

Done.

http://codereview.chromium.org/8801012/diff/1/chrome/browser/renderer_host/ch...
File chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
(right):

http://codereview.chromium.org/8801012/diff/1/chrome/browser/renderer_host/ch...
chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc:36:
#include "content/public/common/referrer.h"
On 2011/12/05 18:03:29, cbentzel wrote:
> Why does this need to be included? Looks like forward declaration is
sufficient.

Done.

[cbentzel, 2011-12-05 21:09:20]

LGTM

[commit-bot: I haz the power, 2011-12-05 21:58:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/8801012/2002

[commit-bot: I haz the power, 2011-12-06 00:17:53]

Change committed as 113070