Revert of Log a non-crash dump for 10% of Spitzer pipeline decode failures.

Revert of Log a non-crash dump for 10% of Spitzer pipeline decode failures. (patchset #1 id:1 of https://codereview.chromium.org/1808473002/ )

Reason for revert:
Should have been reverted before M-51 branch, but forgot.

Original issue's description:
> Log a non-crash dump for 10% of Spitzer pipeline decode failures.
>
> Spitzer is currently enabled as a 40% experiment on dev (soon on
> beta as well), but we have abnormally high video-only decode
> failure rates, so add some logging to try and track this down.
>
> ~10% of the time, this change will log the 'subresource_url' crash
> key as a non-crash dump when a decode error is encountered on
> Android where Spitzer is enabled.
>
> This will allow us to investigate and triage these crashes. We're
> already logging this key for normal crash dumps during DoLoad().
>
> This is a temporary change which will be removed shortly after
> landing once any issues are identified.
>
> BUG=595076
> TEST=none
>
> Committed: https://crrev.com/6e3901e3f6240d8660cb039fdcbb1dd6f75121f6
> Cr-Commit-Position: refs/heads/master@{#381532}

TBR=rsesek@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=595076
Committed: https://crrev.com/a5db39dd84b2fe72eb3260c35759a733bfa3df70
Cr-Commit-Position: refs/heads/master@{#386879}

[DaleCurtis, 2016-04-12 23:42:16]

Created Revert of Log a non-crash dump for 10% of Spitzer pipeline decode
failures.

[commit-bot: I haz the power, 2016-04-12 23:42:32]

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

[commit-bot: I haz the power, 2016-04-12 23:58:54]

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

[commit-bot: I haz the power, 2016-04-12 23:58:55]

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

[DaleCurtis, 2016-04-13 00:42:02]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2016-04-13 00:42:28]

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

[commit-bot: I haz the power, 2016-04-13 00:56:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-13 00:57:35]

Description was changed from

==========
Revert of Log a non-crash dump for 10% of Spitzer pipeline decode failures.
(patchset #1 id:1 of https://codereview.chromium.org/1808473002/ )

Reason for revert:
Should have been reverted before M-51 branch, but forgot.

Original issue's description:
> Log a non-crash dump for 10% of Spitzer pipeline decode failures.
> 
> Spitzer is currently enabled as a 40% experiment on dev (soon on
> beta as well), but we have abnormally high video-only decode
> failure rates, so add some logging to try and track this down.
> 
> ~10% of the time, this change will log the 'subresource_url' crash
> key as a non-crash dump when a decode error is encountered on
> Android where Spitzer is enabled.
> 
> This will allow us to investigate and triage these crashes. We're
> already logging this key for normal crash dumps during DoLoad().
> 
> This is a temporary change which will be removed shortly after
> landing once any issues are identified.
> 
> BUG=595076
> TEST=none
> 
> Committed: https://crrev.com/6e3901e3f6240d8660cb039fdcbb1dd6f75121f6
> Cr-Commit-Position: refs/heads/master@{#381532}

TBR=rsesek@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=595076
==========

to

==========
Revert of Log a non-crash dump for 10% of Spitzer pipeline decode failures.
(patchset #1 id:1 of https://codereview.chromium.org/1808473002/ )

Reason for revert:
Should have been reverted before M-51 branch, but forgot.

Original issue's description:
> Log a non-crash dump for 10% of Spitzer pipeline decode failures.
>
> Spitzer is currently enabled as a 40% experiment on dev (soon on
> beta as well), but we have abnormally high video-only decode
> failure rates, so add some logging to try and track this down.
>
> ~10% of the time, this change will log the 'subresource_url' crash
> key as a non-crash dump when a decode error is encountered on
> Android where Spitzer is enabled.
>
> This will allow us to investigate and triage these crashes. We're
> already logging this key for normal crash dumps during DoLoad().
>
> This is a temporary change which will be removed shortly after
> landing once any issues are identified.
>
> BUG=595076
> TEST=none
>
> Committed: https://crrev.com/6e3901e3f6240d8660cb039fdcbb1dd6f75121f6
> Cr-Commit-Position: refs/heads/master@{#381532}

TBR=rsesek@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=595076

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

[commit-bot: I haz the power, 2016-04-13 00:57:36]

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