Remove unused linked_ptr inclusion

Remove unused linked_ptr inclusion

TBR=jam@chromium.org
BUG=556939
Committed: https://crrev.com/090622c5da065ed0fa7c4db26719b7eb58198377
Cr-Commit-Position: refs/heads/master@{#437526}

[dcheng, 2016-12-07 18:27:13]

LGTM, but I'm not an OWNER for these files.

[Sunny, 2016-12-07 18:31:08]

On 2016/12/07 18:27:13, dcheng wrote:
> LGTM, but I'm not an OWNER for these files.

Thanks for the quick response, can I try the CQ dry run with your LGTM before
adding other reviewers?

[dcheng, 2016-12-07 18:36:48]

This is OK to TBR, but you should add the other reviewers, update the CL
description to have a TBR=<other reviewers>, and then CQ =)

[Sunny, 2016-12-08 05:23:30]

Description was changed from

==========
Remove unused linked_ptr inclusion

R=dcheng@chromium.org
BUG=556939
==========

to

==========
Remove unused linked_ptr inclusion

TBR=jam@chromium.org
COMMIT=false
BUG=556939
==========

[Sunny, 2016-12-08 05:23:31]

ratsunny@gmail.com changed reviewers:
+ jam@chromium.org

[Sunny, 2016-12-08 05:24:20]

The CQ bit was checked by ratsunny@gmail.com

[commit-bot: I haz the power, 2016-12-08 05:24:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-08 05:24:33]

COMMIT=false detected. CQ is abandoning the patch.

[Sunny, 2016-12-08 05:29:32]

On 2016/12/07 18:36:48, dcheng wrote:
> This is OK to TBR, but you should add the other reviewers, update the CL
> description to have a TBR=<other reviewers>, and then CQ =)

Hi, dcheng

I've set TBR to jam, and set commit bit to false to prevent commit accidentally.
But CQ refused to try the patch.
If I remove COMMIT=false and CQ passed, it will be commited without jam's
review, is it acceptable?

Thanks

[Sunny, 2016-12-08 05:35:03]

On 2016/12/07 18:36:48, dcheng wrote:
> This is OK to TBR, but you should add the other reviewers, update the CL
> description to have a TBR=<other reviewers>, and then CQ =)

And to my surprise, there are two trybots (linux_chromium_chromeos_rel_ng and 
linux_chromium_rel_ng) is trying the patch now even "CQ is abandoning the
patch", is it by design?

[jam, 2016-12-08 17:02:48]

lgtm

[dcheng, 2016-12-08 22:10:29]

On 2016/12/08 05:29:32, Sunny wrote:
> On 2016/12/07 18:36:48, dcheng wrote:
> > This is OK to TBR, but you should add the other reviewers, update the CL
> > description to have a TBR=<other reviewers>, and then CQ =)
> 
> Hi, dcheng
> 
> I've set TBR to jam, and set commit bit to false to prevent commit
accidentally.
> But CQ refused to try the patch.
> If I remove COMMIT=false and CQ passed, it will be commited without jam's
> review, is it acceptable?
> 
> Thanks

TBR = to be reviewed, and it means that it's OK to land the CL without getting
OWNERS approval for everything (because a non-OWNER has reviewed). Usually,
simple refactorings / cleanups like this are OK to land TBR, which means you
don't need COMMIT=false in the description.

[Sunny, 2016-12-09 09:35:23]

Description was changed from

==========
Remove unused linked_ptr inclusion

TBR=jam@chromium.org
COMMIT=false
BUG=556939
==========

to

==========
Remove unused linked_ptr inclusion

TBR=jam@chromium.org
BUG=556939
==========

[Sunny, 2016-12-09 09:37:48]

The CQ bit was checked by ratsunny@gmail.com

[commit-bot: I haz the power, 2016-12-09 09:38:04]

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, 2016-12-09 11:12:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-09 11:12:49]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Sunny, 2016-12-09 12:56:27]

The CQ bit was checked by ratsunny@gmail.com

[commit-bot: I haz the power, 2016-12-09 12:56:36]

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, 2016-12-09 13:38:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481288187143320, "parent_rev":
"2d8f6d96dae417f74b95f2f2f0e3b7ff477f9d75", "commit_rev":
"df436c96c3a565e988b8810563560efed83e3e89"}

[commit-bot: I haz the power, 2016-12-09 13:38:58]

Description was changed from

==========
Remove unused linked_ptr inclusion

TBR=jam@chromium.org
BUG=556939
==========

to

==========
Remove unused linked_ptr inclusion

TBR=jam@chromium.org
BUG=556939

Review-Url: https://codereview.chromium.org/2546183004
==========

[commit-bot: I haz the power, 2016-12-09 13:39:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-09 13:40:25]

Description was changed from

==========
Remove unused linked_ptr inclusion

TBR=jam@chromium.org
BUG=556939

Review-Url: https://codereview.chromium.org/2546183004
==========

to

==========
Remove unused linked_ptr inclusion

TBR=jam@chromium.org
BUG=556939

Committed: https://crrev.com/090622c5da065ed0fa7c4db26719b7eb58198377
Cr-Commit-Position: refs/heads/master@{#437526}
==========

[commit-bot: I haz the power, 2016-12-09 13:40:26]

Patchset 1 (id:??) landed as
https://crrev.com/090622c5da065ed0fa7c4db26719b7eb58198377
Cr-Commit-Position: refs/heads/master@{#437526}