Revert of Ensure that builders that use goma strip absolute paths by default.

Revert of Ensure that builders that use goma strip absolute paths by default. (patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )

Reason for revert:
Reverting in case this is related to the goma errors in https://crbug.com/716089

Original issue's description:
> Ensure that builders that use goma strip absolute paths by default.
> 
> This CL tweaks the MB configs so that any builder that is using goma
> also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> This helps ensure that we get the best goma cache hit rate possible.
> We don't just make the default equal to use_goma=true because doing
> so may make it harder for devs to debug binaries, and we want devs
> to be able to both debug binaries and use goma by default; the
> tradeoff is that their builds are slightly slower.
> 
> R=thakis@chromium.org, jochen@chromium.org
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2837863005
> Cr-Commit-Position: refs/heads/master@{#467574}
> Committed: https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4e79642b9edc3

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
Review-Url: https://codereview.chromium.org/2843403004
Cr-Commit-Position: refs/heads/master@{#467769}
Committed: https://chromium.googlesource.com/chromium/src/+/999471957dc5cf51048b0b344c58925a2954d452

[Dirk Pranke, 2017-04-27 20:20:32]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2017-04-27 20:20:33]

Created Revert of Ensure that builders that use goma strip absolute paths by
default.

[commit-bot: I haz the power, 2017-04-27 20:20:54]

CQ is trying da patch.

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

[Nico, 2017-04-27 20:21:05]

Description was changed from

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )

Reason for revert:
Reverting in case this is related to the goma errors in crbug.om/716089

Original issue's description:
> Ensure that builders that use goma strip absolute paths by default.
> 
> This CL tweaks the MB configs so that any builder that is using goma
> also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> This helps ensure that we get the best goma cache hit rate possible.
> We don't just make the default equal to use_goma=true because doing
> so may make it harder for devs to debug binaries, and we want devs
> to be able to both debug binaries and use goma by default; the
> tradeoff is that their builds are slightly slower.
> 
> R=thakis@chromium.org, jochen@chromium.org
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2837863005
> Cr-Commit-Position: refs/heads/master@{#467574}
> Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
==========

to

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )

Reason for revert:
Reverting in case this is related to the goma errors in crbug.com/716089

Original issue's description:
> Ensure that builders that use goma strip absolute paths by default.
> 
> This CL tweaks the MB configs so that any builder that is using goma
> also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> This helps ensure that we get the best goma cache hit rate possible.
> We don't just make the default equal to use_goma=true because doing
> so may make it harder for devs to debug binaries, and we want devs
> to be able to both debug binaries and use goma by default; the
> tradeoff is that their builds are slightly slower.
> 
> R=thakis@chromium.org, jochen@chromium.org
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2837863005
> Cr-Commit-Position: refs/heads/master@{#467574}
> Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
==========

[Nico, 2017-04-27 20:21:17]

Description was changed from

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )

Reason for revert:
Reverting in case this is related to the goma errors in crbug.com/716089

Original issue's description:
> Ensure that builders that use goma strip absolute paths by default.
> 
> This CL tweaks the MB configs so that any builder that is using goma
> also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> This helps ensure that we get the best goma cache hit rate possible.
> We don't just make the default equal to use_goma=true because doing
> so may make it harder for devs to debug binaries, and we want devs
> to be able to both debug binaries and use goma by default; the
> tradeoff is that their builds are slightly slower.
> 
> R=thakis@chromium.org, jochen@chromium.org
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2837863005
> Cr-Commit-Position: refs/heads/master@{#467574}
> Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
==========

to

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )

Reason for revert:
Reverting in case this is related to the goma errors in https://crbug.com/716089

Original issue's description:
> Ensure that builders that use goma strip absolute paths by default.
> 
> This CL tweaks the MB configs so that any builder that is using goma
> also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> This helps ensure that we get the best goma cache hit rate possible.
> We don't just make the default equal to use_goma=true because doing
> so may make it harder for devs to debug binaries, and we want devs
> to be able to both debug binaries and use goma by default; the
> tradeoff is that their builds are slightly slower.
> 
> R=thakis@chromium.org, jochen@chromium.org
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2837863005
> Cr-Commit-Position: refs/heads/master@{#467574}
> Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
==========

[commit-bot: I haz the power, 2017-04-27 20:32:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493324432847560, "parent_rev":
"5cf597d4e4e4409bf219d499ff00972fa60fcf0c", "commit_rev":
"999471957dc5cf51048b0b344c58925a2954d452"}

[commit-bot: I haz the power, 2017-04-27 20:34:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493324432847560, "parent_rev":
"5cf597d4e4e4409bf219d499ff00972fa60fcf0c", "commit_rev":
"999471957dc5cf51048b0b344c58925a2954d452"}

[commit-bot: I haz the power, 2017-04-27 20:34:46]

Description was changed from

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )

Reason for revert:
Reverting in case this is related to the goma errors in https://crbug.com/716089

Original issue's description:
> Ensure that builders that use goma strip absolute paths by default.
> 
> This CL tweaks the MB configs so that any builder that is using goma
> also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> This helps ensure that we get the best goma cache hit rate possible.
> We don't just make the default equal to use_goma=true because doing
> so may make it harder for devs to debug binaries, and we want devs
> to be able to both debug binaries and use goma by default; the
> tradeoff is that their builds are slightly slower.
> 
> R=thakis@chromium.org, jochen@chromium.org
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2837863005
> Cr-Commit-Position: refs/heads/master@{#467574}
> Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
==========

to

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )

Reason for revert:
Reverting in case this is related to the goma errors in https://crbug.com/716089

Original issue's description:
> Ensure that builders that use goma strip absolute paths by default.
> 
> This CL tweaks the MB configs so that any builder that is using goma
> also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> This helps ensure that we get the best goma cache hit rate possible.
> We don't just make the default equal to use_goma=true because doing
> so may make it harder for devs to debug binaries, and we want devs
> to be able to both debug binaries and use goma by default; the
> tradeoff is that their builds are slightly slower.
> 
> R=thakis@chromium.org, jochen@chromium.org
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2837863005
> Cr-Commit-Position: refs/heads/master@{#467574}
> Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790

Review-Url: https://codereview.chromium.org/2843403004
Cr-Commit-Position: refs/heads/master@{#467769}
Committed:
https://chromium.googlesource.com/chromium/src/+/999471957dc5cf51048b0b344c58...
==========

[commit-bot: I haz the power, 2017-04-27 20:34:47]

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

[Dirk Pranke, 2017-05-09 00:59:47]

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

The reason for reverting is: Relanding, since this was not a cause of the goma
outage..