RefCounted types should not have public destructors, content/browser part 1

RefCounted types should not have public destructors, content/browser part 1

BUG=123295
TEST=none


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

[Ryan Sleevi, 2012-04-13 09:23:40]

sky: PTAL

[sky, 2012-04-13 15:51:43]

Same comment as other, move implementation to match and make sure clang bots are
ok with inline destructors.

[Ryan Sleevi, 2012-04-27 01:36:32]

Comments addressed, PTAL

[sky, 2012-04-27 15:45:23]

LGTM

http://codereview.chromium.org/10068037/diff/3001/content/browser/renderer_ho...
File content/browser/renderer_host/media/audio_renderer_host_unittest.cc
(right):

http://codereview.chromium.org/10068037/diff/3001/content/browser/renderer_ho...
content/browser/renderer_host/media/audio_renderer_host_unittest.cc:68:
EXPECT_EQ(0u, audio_entries_.size());
nit: empty

[commit-bot: I haz the power, 2012-04-27 19:43:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsleevi@chromium.org/10068037/8001

[commit-bot: I haz the power, 2012-04-27 20:39:20]

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

[commit-bot: I haz the power, 2012-04-27 21:38:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsleevi@chromium.org/10068037/14001

[commit-bot: I haz the power, 2012-04-27 23:40:59]

Change committed as 134395