Clean up ResourceHandler API.

Clean up ResourceHandler API.

1. Move URLRequest member to ResourceHandler base class and add convenience functions for subclasses.
2. Make OnWillRead take a scoped_refptr to avoid leaking the buffer.

BUG=295239
TEST=No behavior change.

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

[Charlie Reis, 2013-10-01 21:11:23]

Darin, can you take a look?  I added convenience functions for all the cases
that the info object was only being used for one thing.  I can add GetChildID
and GetRoutingID as well if you'd like, but using them side-by-side would have a
bit more overhead than just grabbing the info object.

[Charlie Reis, 2013-10-03 18:10:47]

I've updated this CL to include both the URLRequest cleanup and the
scoped_refptr in OnWillStart.  Please take a look when you get a chance.

Thanks!
Charlie

[darin (slow to review), 2013-10-04 22:05:21]

LGTM!!

[commit-bot: I haz the power, 2013-10-04 22:26:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/creis@chromium.org/25536005/13001

[commit-bot: I haz the power, 2013-10-05 00:10:04]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2013-10-07 05:25:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/creis@chromium.org/25536005/13001

[commit-bot: I haz the power, 2013-10-07 06:58:10]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2013-10-07 17:28:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/creis@chromium.org/25536005/45001

[commit-bot: I haz the power, 2013-10-07 19:45:55]

Change committed as 227318