Carry over a user gesture bit for all the non-main frame request types

Carry over a user gesture bit for all the non-main frame request types

Any non-main frame request type could be used to ping and redirect a navigation.
So we should support a user gesture carryover bit for it.

BUG=478972
Committed: https://crrev.com/0044897feca3e12b01ea0f50939b5a5eb33c6046
Cr-Commit-Position: refs/heads/master@{#326146}

[Jaekyun Seok (inactive), 2015-04-21 01:35:48]

jaekyun@chromium.org changed reviewers:
+ jochen@chromium.org, mariakhomenko@chromium.org

[Jaekyun Seok (inactive), 2015-04-21 01:35:48]

mariakhomenko@ and jochen@, please review this CL.

[jochen (gone - plz use gerrit), 2015-04-21 09:44:54]

https://codereview.chromium.org/1056913008/diff/1/chrome/browser/renderer_hos...
File chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
(right):

https://codereview.chromium.org/1056913008/diff/1/chrome/browser/renderer_hos...
chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc:375: }
else if (resource_type == content::RESOURCE_TYPE_XHR ||
why not make any kind of request carry the bit over?

[Jaekyun Seok (inactive), 2015-04-21 10:54:19]

On 2015/04/21 09:44:54, jochen wrote:
>
https://codereview.chromium.org/1056913008/diff/1/chrome/browser/renderer_hos...
> File chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
> (right):
> 
>
https://codereview.chromium.org/1056913008/diff/1/chrome/browser/renderer_hos...
> chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc:375:
}
> else if (resource_type == content::RESOURCE_TYPE_XHR ||
> why not make any kind of request carry the bit over?

That sounds reasonable because any other request could be used for pinging.

I will apply that idea.

[Jaekyun Seok (inactive), 2015-04-21 11:03:37]

PTAL.

https://codereview.chromium.org/1056913008/diff/1/chrome/browser/renderer_hos...
File chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
(right):

https://codereview.chromium.org/1056913008/diff/1/chrome/browser/renderer_hos...
chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc:375: }
else if (resource_type == content::RESOURCE_TYPE_XHR ||
On 2015/04/21 09:44:54, jochen wrote:
> why not make any kind of request carry the bit over?

Done.

[jochen (gone - plz use gerrit), 2015-04-21 11:11:35]

lgtm

[Maria, 2015-04-21 18:03:07]

lgtm

[Jaekyun Seok (inactive), 2015-04-21 21:19:18]

The CQ bit was checked by jaekyun@chromium.org

[commit-bot: I haz the power, 2015-04-21 21:19:35]

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

[commit-bot: I haz the power, 2015-04-21 22:27:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-21 22:29:54]

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