Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu std::unique_ptr.

Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2581663002/ )

Reason for revert:
This change broke Chrome too. It's stats processing wants to make a copy of webrtc's stats mapping, which is no longer possible with std::unique_ptr.

Original issue's description:
> Reland of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2576673002/ )
>
> Reason for revert:
> Downstream project fixed to not make copies while iterating over the stats mapping.
>
> Original issue's description:
> > Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2567143003/ )
> >
> > Reason for revert:
> > The change from rtc::linked_ptr to std::unique_ptr broke a downstream project.
> >
> > Original issue's description:
> > > Delete rtc::linked_ptr. Only use, in statstypes.h, replaced with std::unique_ptr.
> > >
> > > BUG=webrtc:6424
> > >
> > > Committed: https://crrev.com/36f74e55792cae19db8b222c29aa38d6e0eb1225
> > > Cr-Commit-Position: refs/heads/master@{#15588}
> >
> > TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:6424
> >
> > Committed: https://crrev.com/8afbc8cba65d99bb7a0feece8fb3055b144106b1
> > Cr-Commit-Position: refs/heads/master@{#15589}
>
> TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:6424
>
> Committed: https://crrev.com/06035cf53abad80b0525f286a3b81e388cc7ee00
> Cr-Commit-Position: refs/heads/master@{#15627}

TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6424
Committed: https://crrev.com/6a58f33450a46370039cd82537612040a91165b3
Cr-Commit-Position: refs/heads/master@{#15629}

[nisse-webrtc, 2016-12-15 11:54:37]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2016-12-15 11:54:37]

Created Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr.

[commit-bot: I haz the power, 2016-12-15 11:54:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-12-15 11:54:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481802877480390, "parent_rev":
"09a12c8c22304227c40c083d128fb17c806ceee2", "commit_rev":
"e69122175e1c96361b61c5b19d7bc462e8b8a025"}

[commit-bot: I haz the power, 2016-12-15 11:54:50]

Description was changed from

==========
Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2581663002/
)

Reason for revert:
This change broke Chrome too. It's stats processing wants to make a copy of
webrtc's stats mapping, which is no longer possible with std::unique_ptr.

Original issue's description:
> Reland of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2576673002/
)
> 
> Reason for revert:
> Downstream project fixed to not make copies while iterating over the stats
mapping.
> 
> Original issue's description:
> > Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2567143003/
)
> >
> > Reason for revert:
> > The change from rtc::linked_ptr to std::unique_ptr broke a downstream
project.
> >
> > Original issue's description:
> > > Delete rtc::linked_ptr. Only use, in statstypes.h, replaced with
std::unique_ptr.
> > >
> > > BUG=webrtc:6424
> > >
> > > Committed: https://crrev.com/36f74e55792cae19db8b222c29aa38d6e0eb1225
> > > Cr-Commit-Position: refs/heads/master@{#15588}
> >
> > TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:6424
> >
> > Committed: https://crrev.com/8afbc8cba65d99bb7a0feece8fb3055b144106b1
> > Cr-Commit-Position: refs/heads/master@{#15589}
> 
> TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:6424
> 
> Committed: https://crrev.com/06035cf53abad80b0525f286a3b81e388cc7ee00
> Cr-Commit-Position: refs/heads/master@{#15627}

TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6424
==========

to

==========
Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2581663002/
)

Reason for revert:
This change broke Chrome too. It's stats processing wants to make a copy of
webrtc's stats mapping, which is no longer possible with std::unique_ptr.

Original issue's description:
> Reland of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2576673002/
)
> 
> Reason for revert:
> Downstream project fixed to not make copies while iterating over the stats
mapping.
> 
> Original issue's description:
> > Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2567143003/
)
> >
> > Reason for revert:
> > The change from rtc::linked_ptr to std::unique_ptr broke a downstream
project.
> >
> > Original issue's description:
> > > Delete rtc::linked_ptr. Only use, in statstypes.h, replaced with
std::unique_ptr.
> > >
> > > BUG=webrtc:6424
> > >
> > > Committed: https://crrev.com/36f74e55792cae19db8b222c29aa38d6e0eb1225
> > > Cr-Commit-Position: refs/heads/master@{#15588}
> >
> > TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:6424
> >
> > Committed: https://crrev.com/8afbc8cba65d99bb7a0feece8fb3055b144106b1
> > Cr-Commit-Position: refs/heads/master@{#15589}
> 
> TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:6424
> 
> Committed: https://crrev.com/06035cf53abad80b0525f286a3b81e388cc7ee00
> Cr-Commit-Position: refs/heads/master@{#15627}

TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6424

Review-Url: https://codereview.webrtc.org/2579753002
==========

[commit-bot: I haz the power, 2016-12-15 11:54:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 11:54:57]

Description was changed from

==========
Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2581663002/
)

Reason for revert:
This change broke Chrome too. It's stats processing wants to make a copy of
webrtc's stats mapping, which is no longer possible with std::unique_ptr.

Original issue's description:
> Reland of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2576673002/
)
> 
> Reason for revert:
> Downstream project fixed to not make copies while iterating over the stats
mapping.
> 
> Original issue's description:
> > Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2567143003/
)
> >
> > Reason for revert:
> > The change from rtc::linked_ptr to std::unique_ptr broke a downstream
project.
> >
> > Original issue's description:
> > > Delete rtc::linked_ptr. Only use, in statstypes.h, replaced with
std::unique_ptr.
> > >
> > > BUG=webrtc:6424
> > >
> > > Committed: https://crrev.com/36f74e55792cae19db8b222c29aa38d6e0eb1225
> > > Cr-Commit-Position: refs/heads/master@{#15588}
> >
> > TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:6424
> >
> > Committed: https://crrev.com/8afbc8cba65d99bb7a0feece8fb3055b144106b1
> > Cr-Commit-Position: refs/heads/master@{#15589}
> 
> TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:6424
> 
> Committed: https://crrev.com/06035cf53abad80b0525f286a3b81e388cc7ee00
> Cr-Commit-Position: refs/heads/master@{#15627}

TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6424

Review-Url: https://codereview.webrtc.org/2579753002
==========

to

==========
Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2581663002/
)

Reason for revert:
This change broke Chrome too. It's stats processing wants to make a copy of
webrtc's stats mapping, which is no longer possible with std::unique_ptr.

Original issue's description:
> Reland of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2576673002/
)
>
> Reason for revert:
> Downstream project fixed to not make copies while iterating over the stats
mapping.
>
> Original issue's description:
> > Revert of Delete rtc::linked_ptr. Only use, in statstypes.h, replaced bu
std::unique_ptr. (patchset #1 id:1 of https://codereview.webrtc.org/2567143003/
)
> >
> > Reason for revert:
> > The change from rtc::linked_ptr to std::unique_ptr broke a downstream
project.
> >
> > Original issue's description:
> > > Delete rtc::linked_ptr. Only use, in statstypes.h, replaced with
std::unique_ptr.
> > >
> > > BUG=webrtc:6424
> > >
> > > Committed: https://crrev.com/36f74e55792cae19db8b222c29aa38d6e0eb1225
> > > Cr-Commit-Position: refs/heads/master@{#15588}
> >
> > TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:6424
> >
> > Committed: https://crrev.com/8afbc8cba65d99bb7a0feece8fb3055b144106b1
> > Cr-Commit-Position: refs/heads/master@{#15589}
>
> TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:6424
>
> Committed: https://crrev.com/06035cf53abad80b0525f286a3b81e388cc7ee00
> Cr-Commit-Position: refs/heads/master@{#15627}

TBR=solenberg@webrtc.org,pthatcher@webrtc.org,hta@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6424

Committed: https://crrev.com/6a58f33450a46370039cd82537612040a91165b3
Cr-Commit-Position: refs/heads/master@{#15629}
==========

[commit-bot: I haz the power, 2016-12-15 11:54:58]

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

[nisse-webrtc, 2017-01-18 09:21:44]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2641793002/ by nisse@webrtc.org.

The reason for reverting is: Intend to switch from unique_ptr to scoped_refptr,
and then reland..

[the sun, 2017-01-18 09:45:33]

lgtm