Revert of [devil] Use /data/local/tmp for the command line on eng + userdebug builds.

Revert of [devil] Use /data/local/tmp for the command line on eng + userdebug builds. (patchset #2 id:20001 of https://codereview.chromium.org/2275863002/ )

Reason for revert:
Broke telemetry tests still using the old command line path

Original issue's description:
> [devil] Use /data/local/tmp for the command line on eng + userdebug builds.
> 
> Matches https://codereview.chromium.org/2160243002
> 
> BUG=chromium:640228
> 
> Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/ae022b60c580ffbba455d620aab295372ac5ab0d

TBR=rnephew@chromium.org,mikecase@chromium.org,jbudorick@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:640228, chromium:641497
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/aa7c58eb04599138bc97f93245c1acf3c5b81f85

[perezju, 2016-08-31 12:33:04]

Created Revert of [devil] Use /data/local/tmp for the command line on eng +
userdebug builds.

[perezju, 2016-08-31 12:33:31]

On 2016/08/31 12:33:04, perezju wrote:
> Created Revert of [devil] Use /data/local/tmp for the command line on eng +
> userdebug builds.

I created the rever to try out locally and confirm whether this fixes the issue.

[jbudorick, 2016-08-31 12:58:33]

On 2016/08/31 12:33:31, perezju wrote:
> On 2016/08/31 12:33:04, perezju wrote:
> > Created Revert of [devil] Use /data/local/tmp for the command line on eng +
> > userdebug builds.
> 
> I created the rever to try out locally and confirm whether this fixes the
issue.

I take it that you're investigating
https://bugs.chromium.org/p/chromium/issues/detail?id=641497 ?

[perezju, 2016-08-31 13:00:26]

On 2016/08/31 12:58:33, jbudorick wrote:
> On 2016/08/31 12:33:31, perezju wrote:
> > On 2016/08/31 12:33:04, perezju wrote:
> > > Created Revert of [devil] Use /data/local/tmp for the command line on eng
+
> > > userdebug builds.
> > 
> > I created the rever to try out locally and confirm whether this fixes the
> issue.
> 
> I take it that you're investigating
> https://bugs.chromium.org/p/chromium/issues/detail?id=641497 ?

Yep, and just confirmed locally that this fixes the issue. Are you happy with
landing this revert?

[perezju, 2016-08-31 13:01:11]

Description was changed from

==========
Revert of [devil] Use /data/local/tmp for the command line on eng + userdebug
builds. (patchset #2 id:20001 of https://codereview.chromium.org/2275863002/ )

Reason for revert:
Broke telemetry tests still using the old command line path

Original issue's description:
> [devil] Use /data/local/tmp for the command line on eng + userdebug builds.
> 
> Matches https://codereview.chromium.org/2160243002
> 
> BUG=chromium:640228
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=rnephew@chromium.org,mikecase@chromium.org,jbudorick@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:640228
==========

to

==========
Revert of [devil] Use /data/local/tmp for the command line on eng + userdebug
builds. (patchset #2 id:20001 of https://codereview.chromium.org/2275863002/ )

Reason for revert:
Broke telemetry tests still using the old command line path

Original issue's description:
> [devil] Use /data/local/tmp for the command line on eng + userdebug builds.
> 
> Matches https://codereview.chromium.org/2160243002
> 
> BUG=chromium:640228
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=rnephew@chromium.org,mikecase@chromium.org,jbudorick@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:640228,chromium:641497
==========

[jbudorick, 2016-08-31 13:02:57]

On 2016/08/31 13:00:26, perezju wrote:
> On 2016/08/31 12:58:33, jbudorick wrote:
> > On 2016/08/31 12:33:31, perezju wrote:
> > > On 2016/08/31 12:33:04, perezju wrote:
> > > > Created Revert of [devil] Use /data/local/tmp for the command line on
eng
> +
> > > > userdebug builds.
> > > 
> > > I created the rever to try out locally and confirm whether this fixes the
> > issue.
> > 
> > I take it that you're investigating
> > https://bugs.chromium.org/p/chromium/issues/detail?id=641497 ?
> 
> Yep, and just confirmed locally that this fixes the issue. Are you happy with
> landing this revert?

Yeah, that's fine with me.

[jbudorick, 2016-08-31 13:03:20]

On 2016/08/31 13:02:57, jbudorick wrote:
> On 2016/08/31 13:00:26, perezju wrote:
> > On 2016/08/31 12:58:33, jbudorick wrote:
> > > On 2016/08/31 12:33:31, perezju wrote:
> > > > On 2016/08/31 12:33:04, perezju wrote:
> > > > > Created Revert of [devil] Use /data/local/tmp for the command line on
> eng
> > +
> > > > > userdebug builds.
> > > > 
> > > > I created the rever to try out locally and confirm whether this fixes
the
> > > issue.
> > > 
> > > I take it that you're investigating
> > > https://bugs.chromium.org/p/chromium/issues/detail?id=641497 ?
> > 
> > Yep, and just confirmed locally that this fixes the issue. Are you happy
with
> > landing this revert?
> 
> Yeah, that's fine with me.

lgtm, that is

[perezju, 2016-08-31 13:03:39]

The CQ bit was checked by perezju@chromium.org

[commit-bot: I haz the power, 2016-08-31 13:03:49]

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-08-31 13:25:26]

Description was changed from

==========
Revert of [devil] Use /data/local/tmp for the command line on eng + userdebug
builds. (patchset #2 id:20001 of https://codereview.chromium.org/2275863002/ )

Reason for revert:
Broke telemetry tests still using the old command line path

Original issue's description:
> [devil] Use /data/local/tmp for the command line on eng + userdebug builds.
> 
> Matches https://codereview.chromium.org/2160243002
> 
> BUG=chromium:640228
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=rnephew@chromium.org,mikecase@chromium.org,jbudorick@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:640228,chromium:641497
==========

to

==========
Revert of [devil] Use /data/local/tmp for the command line on eng + userdebug
builds. (patchset #2 id:20001 of https://codereview.chromium.org/2275863002/ )

Reason for revert:
Broke telemetry tests still using the old command line path

Original issue's description:
> [devil] Use /data/local/tmp for the command line on eng + userdebug builds.
> 
> Matches https://codereview.chromium.org/2160243002
> 
> BUG=chromium:640228
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=rnephew@chromium.org,mikecase@chromium.org,jbudorick@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:640228,chromium:641497

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-08-31 13:25:27]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...