Rename URLFetcherImpl::Core::TempFileWriter to FileWriter

Rename URLFetcherImpl::Core::TempFileWriter to FileWriter

URLFetcher is going to have a new method 'SaveResponseToFile'.
FileWriter is going to manage both temporary and ordinary files.

BUG=chromium-os:26971
TEST=content_unittests --gtest_filter="URLFetcher*"


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124846

[hashimoto, 2012-03-02 08:19:08]

This patch is a preparation before adding URLFetcher:: SaveResponseToFile

[willchan no longer on Chromium, 2012-03-02 19:48:22]

http://codereview.chromium.org/9580003/diff/5002/content/common/net/url_fetch...
File content/common/net/url_fetcher_impl.cc (right):

http://codereview.chromium.org/9580003/diff/5002/content/common/net/url_fetch...
content/common/net/url_fetcher_impl.cc:113: void CreateTempFile();
Rename these too?

[hashimoto, 2012-03-03 02:56:37]

http://codereview.chromium.org/9580003/diff/5002/content/common/net/url_fetch...
File content/common/net/url_fetcher_impl.cc (right):

http://codereview.chromium.org/9580003/diff/5002/content/common/net/url_fetch...
content/common/net/url_fetcher_impl.cc:113: void CreateTempFile();
On 2012/03/02 19:48:22, willchan wrote:
> Rename these too?

CreateTempFile is not renamed because it is a method which does something
specific to temporary files.
I am going to add CreateFile as a separate method to create ordinary files. 
(http://codereview.chromium.org/9585009/ is the CL in progress)

[willchan no longer on Chromium, 2012-03-03 03:06:14]

OIC, LGTM

On Fri, Mar 2, 2012 at 6:56 PM, <hashimoto@chromium.org> wrote:

>
> http://codereview.chromium.**org/9580003/diff/5002/content/**
>
common/net/url_fetcher_impl.cc<http://codereview.chromium.org/9580003/diff/5002/content/common/net/url_fetcher_impl.cc>
> File content/common/net/url_**fetcher_impl.cc (right):
>
> http://codereview.chromium.**org/9580003/diff/5002/content/**
>
common/net/url_fetcher_impl.**cc#newcode113<http://codereview.chromium.org/9580003/diff/5002/content/common/net/url_fetcher_impl.cc#newcode113>
> content/common/net/url_**fetcher_impl.cc:113: void CreateTempFile();
> On 2012/03/02 19:48:22, willchan wrote:
>
>> Rename these too?
>>
>
> CreateTempFile is not renamed because it is a method which does
> something specific to temporary files.
> I am going to add CreateFile as a separate method to create ordinary
> files. 
(http://codereview.chromium.**org/9585009/<http://codereview.chromium.org/9585...
the CL in progress)
>
>
http://codereview.chromium.**org/9580003/<http://codereview.chromium.org/9580...
>

[commit-bot: I haz the power, 2012-03-03 03:30:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hashimoto@chromium.org/9580003/5002

[commit-bot: I haz the power, 2012-03-03 08:51:47]

Change committed as 124846