PPAPI: Add an API for setting a custom user agent for URL requests.

PPAPI: Add an API for setting a custom user agent for URL requests.

This doesn't hook it up yet.

Pepper Flash needs this for RTMPT.

BUG=134615
TEST=not really (except for trivial URL request test)

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

[brettw, 2012-07-10 04:54:45]

https://chromiumcodereview.appspot.com/10762017/diff/1/ppapi/api/ppb_url_requ...
File ppapi/api/ppb_url_request_info.idl (right):

https://chromiumcodereview.appspot.com/10762017/diff/1/ppapi/api/ppb_url_requ...
ppapi/api/ppb_url_request_info.idl:152: * to set a custom content transfer
encoding; if given to a loader without
transfer encoding -> user agent

https://chromiumcodereview.appspot.com/10762017/diff/1/ppapi/shared_impl/ppb_...
File ppapi/shared_impl/ppb_url_request_info_shared.h (right):

https://chromiumcodereview.appspot.com/10762017/diff/1/ppapi/shared_impl/ppb_...
ppapi/shared_impl/ppb_url_request_info_shared.h:79: bool has_custom_user_agent;
Are we ever going to allow an empty user agent? Can we just use the string being
empty or not?

[viettrungluu, 2012-07-10 13:30:52]

https://chromiumcodereview.appspot.com/10762017/diff/1/ppapi/shared_impl/ppb_...
File ppapi/shared_impl/ppb_url_request_info_shared.h (right):

https://chromiumcodereview.appspot.com/10762017/diff/1/ppapi/shared_impl/ppb_...
ppapi/shared_impl/ppb_url_request_info_shared.h:79: bool has_custom_user_agent;
On 2012/07/10 04:54:45, brettw wrote:
> Are we ever going to allow an empty user agent? Can we just use the string
being
> empty or not?

Presumably an empty user agent would mean "no user agent header". Really, the
point of this is to keep it consistent with the other custom_foos (and the
conventions for setting them).

(Sadly, std::string doesn't allow for a distinction between null/undefined and
empty.)

[brettw, 2012-07-10 14:06:34]

lgtm