Move RefCountedMemory class to base namespace.

Move RefCountedMemory Class to Base namespace

BUG=none
TEST=compiles chrome, browser_tests and ui_tests

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

[simonhong_, 2012-04-30 01:17:41]

Dear joth, erg
Please review this issue.
Thank you.

[Elliot Glaysher, 2012-05-01 19:08:49]

I read the entire patch. Other than the one nit in ref_counted_memory.h, lgtm.

http://codereview.chromium.org/10272004/diff/3009/base/memory/ref_counted_mem...
File base/memory/ref_counted_memory.h (right):

http://codereview.chromium.org/10272004/diff/3009/base/memory/ref_counted_mem...
base/memory/ref_counted_memory.h:19: namespace base {
This goes above the comment; this comment describes the class.

[simonhong_, 2012-05-01 19:17:25]

http://codereview.chromium.org/10272004/diff/3009/base/memory/ref_counted_mem...
File base/memory/ref_counted_memory.h (right):

http://codereview.chromium.org/10272004/diff/3009/base/memory/ref_counted_mem...
base/memory/ref_counted_memory.h:19: namespace base {
On 2012/05/01 19:08:49, Elliot Glaysher wrote:
> This goes above the comment; this comment describes the class.

Done.

[simonhong_, 2012-05-01 19:28:13]

Dear brettw, ben
Please review this issue.

brettw:
  base/*
  chrome/*
ben:
  ui/*

Thank you.

[Ben Goodger (Google), 2012-05-01 19:32:37]

LGTM for ui/

[brettw, 2012-05-01 21:26:22]

base LGTM

[simonhong_, 2012-05-01 23:10:34]

Could I check commit box?
Or need more stamp sub dirs in the chrome/ ?
If more reviewer are needed, I will find and request.
I think requesting all under chrome/ to bretts is not suitable
because this issue has too many files.
Of course, this is so trivial but time consuming for brettw.

[Elliot Glaysher, 2012-05-02 00:23:02]

On 2012/05/01 23:10:34, Simon Hong wrote:
> Could I check commit box?
> Or need more stamp sub dirs in the chrome/ ?
> If more reviewer are needed, I will find and request.
> I think requesting all under chrome/ to bretts is not suitable
> because this issue has too many files.
> Of course, this is so trivial but time consuming for brettw.

I just checked the commit box. Either it will pass (everything is therefore OK),
or it will fail (and list the directories where you need reviewers).

[commit-bot: I haz the power, 2012-05-02 00:23:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/simon.hong81@gmail.com/10272004/11013

[commit-bot: I haz the power, 2012-05-02 00:50:58]

Try job failure for 10272004-11013 (retry) on mac_rel for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-05-02 20:49:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/simon.hong81@gmail.com/10272004/26001

[commit-bot: I haz the power, 2012-05-02 22:37:46]

Change committed as 135011