Minor cleanups.

Minor cleanups.

BUG=none
TEST=none

[thakis, 2012-03-23 01:15:17]

Addresses comments from https://codereview.appspot.com/5864055/

The thread screw-up is in chromium code, I'll cc you on the CL where I fix that
(very likely tomorrow).

[willchan, 2012-03-23 01:28:53]

LGTM

On Thu, Mar 22, 2012 at 6:15 PM, <thakis@chromium.org> wrote:

> Reviewers: willchan,
>
> Message:
> Addresses comments from
https://codereview.appspot.**com/5864055/<https://codereview.appspot.com/5864...
>
> The thread screw-up is in chromium code, I'll cc you on the CL where I
> fix that (very likely tomorrow).
>
> Description:
> Minor cleanups.
>
> BUG=none
> TEST=none
>
>
> Please review this at
https://codereview.appspot.**com/5882054/<https://codereview.appspot.com/5882...
>
> Affected files:
>  M lib/financial_ping.cc
>  M rlz.gyp
>
>
> Index: lib/financial_ping.cc
> diff --git a/lib/financial_ping.cc b/lib/financial_ping.cc
> index 1c278bc10b20532bde742b14406f39**13427b1a0f..**
> 28ecd96f5964c61278da2353437e1a**6d0efbaff7 100644
> --- a/lib/financial_ping.cc
> +++ b/lib/financial_ping.cc
> @@ -278,8 +278,8 @@ bool FinancialPing::PingServer(**const char* request,
> std::string* response) {
>       GURL(url), content::URLFetcher::GET, &delegate));
>
>   fetcher->SetLoadFlags(net::**LOAD_DISABLE_CACHE |
> -                        net::LOAD_DO_NOT_SEND_AUTH_**DATA |  // XXX?
> -                        net::LOAD_DO_NOT_PROMPT_FOR_**LOGIN |  // XXX?
> +                        net::LOAD_DO_NOT_SEND_AUTH_**DATA |
> +                        net::LOAD_DO_NOT_PROMPT_FOR_**LOGIN |
>                         net::LOAD_DO_NOT_SEND_COOKIES |
>                         net::LOAD_DO_NOT_SAVE_COOKIES)**;
>
> Index: rlz.gyp
> diff --git a/rlz.gyp b/rlz.gyp
> index 12b6d2db73265a61faafccbaccfd93**46df522169..**
> e1aae72488ca35c725c9b881c91b4e**ece6afd35b 100644
> --- a/rlz.gyp
> +++ b/rlz.gyp
> @@ -81,8 +81,8 @@
>           ],
>           'dependencies': [
>             '../build/temp_gyp/googleurl.**gyp:googleurl',
> -            # TODO: This can change to just net once URLFetcher has moved
> -            # to net, http://118220
> +            '../net/net.gyp:net',
> +            # TODO: Remove this once URLFetcher has moved to net,
> http://118220
>             '../content/content.gyp:**content_common',
>           ],
>         }, {
>
>
>