Revert of Remove Pass() on Android.

Revert of Remove Pass() on Android. (patchset #1 id:1 of https://codereview.chromium.org/1559603002/ )

Reason for revert:
This breaks compilation, as there are still some callsites of Pass() left.

See Android Cronet Data Reduction Proxy Builder build #961 and onwards: https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Data%20Reduction%20Proxy%20Builder/builds/961/steps/compile/logs/stdio

Original issue's description:
> Remove Pass() on Android.
>
> Pass() is a legacy of the original C++03 move emulation implementation.
> New code should use std::move from <utility>.
>
> ❆(੭ु ◜◡‾)੭ु⁾☃❆
>
> BUG=557422
> R=thestig@chromium.org
>
> Committed: https://crrev.com/92469c8c4b7da052273dee1a31308663d1dd1e3d
> Cr-Commit-Position: refs/heads/master@{#367251}

TBR=thestig@chromium.org,dcheng@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557422
Committed: https://crrev.com/f737edcf768bad1883164ac1c2270df670c7f6a9
Cr-Commit-Position: refs/heads/master@{#367811}

[msramek, 2016-01-06 10:21:58]

Created Revert of Remove Pass() on Android.

[commit-bot: I haz the power, 2016-01-06 10:22:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1564763002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1564763002/1

[commit-bot: I haz the power, 2016-01-06 10:23:11]

Description was changed from

==========
Revert of Remove Pass() on Android. (patchset #1 id:1 of
https://codereview.chromium.org/1559603002/ )

Reason for revert:
This breaks compilation, as there are still some callsites of Pass() left.

See Android Cronet Data Reduction Proxy Builder build #961 and onwards:
https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Dat...


Original issue's description:
> Remove Pass() on Android.
> 
> Pass() is a legacy of the original C++03 move emulation implementation.
> New code should use std::move from <utility>.
> 
> ❆(੭ु ◜◡‾)੭ु⁾☃❆
> 
> BUG=557422
> R=thestig@chromium.org
> 
> Committed: https://crrev.com/92469c8c4b7da052273dee1a31308663d1dd1e3d
> Cr-Commit-Position: refs/heads/master@{#367251}

TBR=thestig@chromium.org,dcheng@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557422
==========

to

==========
Revert of Remove Pass() on Android. (patchset #1 id:1 of
https://codereview.chromium.org/1559603002/ )

Reason for revert:
This breaks compilation, as there are still some callsites of Pass() left.

See Android Cronet Data Reduction Proxy Builder build #961 and onwards:
https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Dat...


Original issue's description:
> Remove Pass() on Android.
> 
> Pass() is a legacy of the original C++03 move emulation implementation.
> New code should use std::move from <utility>.
> 
> ❆(੭ु ◜◡‾)੭ु⁾☃❆
> 
> BUG=557422
> R=thestig@chromium.org
> 
> Committed: https://crrev.com/92469c8c4b7da052273dee1a31308663d1dd1e3d
> Cr-Commit-Position: refs/heads/master@{#367251}

TBR=thestig@chromium.org,dcheng@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557422
==========

[commit-bot: I haz the power, 2016-01-06 10:23:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-06 10:24:58]

Description was changed from

==========
Revert of Remove Pass() on Android. (patchset #1 id:1 of
https://codereview.chromium.org/1559603002/ )

Reason for revert:
This breaks compilation, as there are still some callsites of Pass() left.

See Android Cronet Data Reduction Proxy Builder build #961 and onwards:
https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Dat...


Original issue's description:
> Remove Pass() on Android.
> 
> Pass() is a legacy of the original C++03 move emulation implementation.
> New code should use std::move from <utility>.
> 
> ❆(੭ु ◜◡‾)੭ु⁾☃❆
> 
> BUG=557422
> R=thestig@chromium.org
> 
> Committed: https://crrev.com/92469c8c4b7da052273dee1a31308663d1dd1e3d
> Cr-Commit-Position: refs/heads/master@{#367251}

TBR=thestig@chromium.org,dcheng@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557422
==========

to

==========
Revert of Remove Pass() on Android. (patchset #1 id:1 of
https://codereview.chromium.org/1559603002/ )

Reason for revert:
This breaks compilation, as there are still some callsites of Pass() left.

See Android Cronet Data Reduction Proxy Builder build #961 and onwards:
https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Dat...

Original issue's description:
> Remove Pass() on Android.
>
> Pass() is a legacy of the original C++03 move emulation implementation.
> New code should use std::move from <utility>.
>
> ❆(੭ु ◜◡‾)੭ु⁾☃❆
>
> BUG=557422
> R=thestig@chromium.org
>
> Committed: https://crrev.com/92469c8c4b7da052273dee1a31308663d1dd1e3d
> Cr-Commit-Position: refs/heads/master@{#367251}

TBR=thestig@chromium.org,dcheng@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557422

Committed: https://crrev.com/f737edcf768bad1883164ac1c2270df670c7f6a9
Cr-Commit-Position: refs/heads/master@{#367811}
==========

[commit-bot: I haz the power, 2016-01-06 10:24:59]

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

[dcheng, 2016-01-06 13:57:48]

On 2016/01/06 at 10:24:59, commit-bot wrote:
> Patchset 1 (id:??) landed as
https://crrev.com/f737edcf768bad1883164ac1c2270df670c7f6a9
> Cr-Commit-Position: refs/heads/master@{#367811}

I don't think it's reasonable to revert because:
1) there is no try not coverage
2) it's been broken for 6 days and no one could bother telling me first?

[dcheng, 2016-01-06 13:58:17]

On 2016/01/06 at 13:57:48, dcheng wrote:
> On 2016/01/06 at 10:24:59, commit-bot wrote:
> > Patchset 1 (id:??) landed as
https://crrev.com/f737edcf768bad1883164ac1c2270df670c7f6a9
> > Cr-Commit-Position: refs/heads/master@{#367811}
> 
> I don't think it's reasonable to revert because:
> 1) there is no try not coverage

Try bot

> 2) it's been broken for 6 days and no one could bother telling me first?

[dcheng, 2016-01-06 13:58:18]

On 2016/01/06 at 13:57:48, dcheng wrote:
> On 2016/01/06 at 10:24:59, commit-bot wrote:
> > Patchset 1 (id:??) landed as
https://crrev.com/f737edcf768bad1883164ac1c2270df670c7f6a9
> > Cr-Commit-Position: refs/heads/master@{#367811}
> 
> I don't think it's reasonable to revert because:
> 1) there is no try not coverage

Try bot

> 2) it's been broken for 6 days and no one could bother telling me first?

[dcheng, 2016-01-06 13:58:18]

On 2016/01/06 at 13:57:48, dcheng wrote:
> On 2016/01/06 at 10:24:59, commit-bot wrote:
> > Patchset 1 (id:??) landed as
https://crrev.com/f737edcf768bad1883164ac1c2270df670c7f6a9
> > Cr-Commit-Position: refs/heads/master@{#367811}
> 
> I don't think it's reasonable to revert because:
> 1) there is no try not coverage

Try bot

> 2) it's been broken for 6 days and no one could bother telling me first?

[dcheng, 2016-01-06 14:15:51]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1562043002/ by dcheng@chromium.org.

The reason for reverting is: Relanding because the build config in question is
not covered by any trybots or the default waterfall.

Per the discussion:
1) Keeping this reverted makes things harder to reland, due to the possibility
of mid-air collisions.
2) The remaining failures will be fixed shortly afterwards in a followup patch..