Make RefCountedThreadSafeBase::AddRef and Release available for inlining

Make RefCountedThreadSafeBase::AddRef and Release available for inlining

These are hit frequently, and would benefit from inlining even in non-LTO
builds (see bug).

BUG=728324
Review-Url: https://codereview.chromium.org/2947863003
Cr-Commit-Position: refs/heads/master@{#481091}
Committed: https://chromium.googlesource.com/chromium/src/+/5e3bdec771619fd558e176dafe393bfe1690078a

[hans, 2017-06-21 00:02:20]

hans@chromium.org changed reviewers:
+ dcheng@chromium.org, thakis@chromium.org

[hans, 2017-06-21 00:02:21]

Please take a look.

[hans, 2017-06-21 00:02:46]

The CQ bit was checked by hans@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-21 00:03:14]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Nico, 2017-06-21 00:19:41]

Lgtm

I guess with dchecks off this should at worst be size neutral?

[dcheng, 2017-06-21 00:29:01]

lgtm

[hans, 2017-06-21 00:31:53]

On 2017/06/21 00:19:41, Nico wrote:
> Lgtm
> 
> I guess with dchecks off this should at worst be size neutral?

I haven't measured, but yes the generated code for this should be small. I was
counting on the bots to yell at me otherwise.

[hans, 2017-06-21 01:00:49]

I did an official release build on linux, and it saved about 4 KB :-)

[hans, 2017-06-21 01:00:53]

The CQ bit was unchecked by hans@chromium.org

[hans, 2017-06-21 01:00:57]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2017-06-21 01:01:13]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-21 02:40:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498006857534860, "parent_rev":
"73f022ae44572601cb364871eaba46088c3c2666", "commit_rev":
"5e3bdec771619fd558e176dafe393bfe1690078a"}

[commit-bot: I haz the power, 2017-06-21 02:40:53]

Description was changed from

==========
Make RefCountedThreadSafeBase::AddRef and Release available for inlining

These are hit frequently, and would benefit from inlining even in non-LTO
builds (see bug).

BUG=728324
==========

to

==========
Make RefCountedThreadSafeBase::AddRef and Release available for inlining

These are hit frequently, and would benefit from inlining even in non-LTO
builds (see bug).

BUG=728324

Review-Url: https://codereview.chromium.org/2947863003
Cr-Commit-Position: refs/heads/master@{#481091}
Committed:
https://chromium.googlesource.com/chromium/src/+/5e3bdec771619fd558e176dafe39...
==========

[commit-bot: I haz the power, 2017-06-21 02:40:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5e3bdec771619fd558e176dafe39...