Add tracing for DocumentSubresourceFilter::allowLoad.

Add tracing for DocumentSubresourceFilter::allowLoad.

BUG=609747
Committed: https://crrev.com/e8a730f436de02c715e7169ef906e7d48c2dc73c
Cr-Commit-Position: refs/heads/master@{#406836}

[engedy, 2016-07-19 18:43:27]

Description was changed from

==========
Add tracing for DocumentSubresourceFilter::allowLoad.

BUG=609747
==========

to

==========
Add tracing for DocumentSubresourceFilter::allowLoad.

BUG=609747
==========

[engedy, 2016-07-19 18:43:27]

engedy@chromium.org changed reviewers:
+ melandory@chromium.org, simonhatch@chromium.org

[engedy, 2016-07-19 18:55:16]

@Simon, can you please look at whether this would be the right category? Also,
can you see a less awkward way to record a WebURL here? Not sure if
GURL(resourceUrl).possibly_invalid_spec() would be any nicer.

In any case, I don't want to create a named GURL local variable and reuse that,
as I'd like to include the time it takes to do the KURL -> GURL conversion
exactly once in the measurements.

@Tanja, please review as OWNER.

[shatch, 2016-07-20 16:51:17]

On 2016/07/19 18:55:16, engedy wrote:
> @Simon, can you please look at whether this would be the right category? Also,
> can you see a less awkward way to record a WebURL here? Not sure if
> GURL(resourceUrl).possibly_invalid_spec() would be any nicer.
> 

Both seem fine to me.


> In any case, I don't want to create a named GURL local variable and reuse
that,
> as I'd like to include the time it takes to do the KURL -> GURL conversion
> exactly once in the measurements.
> 
> @Tanja, please review as OWNER.

[engedy, 2016-07-20 19:53:39]

engedy@chromium.org changed reviewers:
- simonhatch@chromium.org

[engedy, 2016-07-20 19:54:03]

Thank you! @Tanja, back to you for OWNER's review.

[melandory, 2016-07-21 12:39:22]

On 2016/07/20 19:54:03, engedy wrote:
> Thank you! @Tanja, back to you for OWNER's review.

lgtm, sorry for delay.

[engedy, 2016-07-21 13:29:03]

On 2016/07/21 12:39:22, melandory wrote:
> On 2016/07/20 19:54:03, engedy wrote:
> > Thank you! @Tanja, back to you for OWNER's review.
> 
> lgtm, sorry for delay.

No worries!

[engedy, 2016-07-21 13:29:08]

The CQ bit was checked by engedy@chromium.org

[commit-bot: I haz the power, 2016-07-21 13:29:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-21 14:02:32]

Description was changed from

==========
Add tracing for DocumentSubresourceFilter::allowLoad.

BUG=609747
==========

to

==========
Add tracing for DocumentSubresourceFilter::allowLoad.

BUG=609747
==========

[commit-bot: I haz the power, 2016-07-21 14:02:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-21 14:04:42]

Description was changed from

==========
Add tracing for DocumentSubresourceFilter::allowLoad.

BUG=609747
==========

to

==========
Add tracing for DocumentSubresourceFilter::allowLoad.

BUG=609747

Committed: https://crrev.com/e8a730f436de02c715e7169ef906e7d48c2dc73c
Cr-Commit-Position: refs/heads/master@{#406836}
==========

[commit-bot: I haz the power, 2016-07-21 14:04:43]

Patchset 1 (id:??) landed as
https://crrev.com/e8a730f436de02c715e7169ef906e7d48c2dc73c
Cr-Commit-Position: refs/heads/master@{#406836}