Revert of QUIC - instrumentation for testing null QuicSpdyStream.

Revert of QUIC - instrumentation for testing null QuicSpdyStream. (patchset #5 id:100001 of https://codereview.chromium.org/1688003005/ )

Reason for revert:
We got the call stack that causes the crash. Reverting this change. Will add the check for nullptr and the unit tests.

Original issue's description:
> QUIC - instrumentation for testing null QuicSpdyStream.
>
> BUG=585591
> R=rch@chromium.org,eroman@chromium.org
>
> Committed: https://crrev.com/100e39060a98afdc80f7a5140f6ba690746261b9
> Cr-Commit-Position: refs/heads/master@{#375804}

TBR=rch@chromium.org,eroman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=585591
Committed: https://crrev.com/45e15292b538db8e1b60f4e0eeea1c0587e3c190
Cr-Commit-Position: refs/heads/master@{#378582}

[ramant (doing other things), 2016-03-01 19:50:48]

Created Revert of QUIC - instrumentation for testing null QuicSpdyStream.

[commit-bot: I haz the power, 2016-03-01 19:51:28]

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

[commit-bot: I haz the power, 2016-03-01 19:54:59]

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

[commit-bot: I haz the power, 2016-03-01 19:54:59]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[eroman, 2016-03-01 20:00:53]

lgtm

[ramant (doing other things), 2016-03-01 20:19:09]

The CQ bit was checked by rtenneti@chromium.org

[ramant (doing other things), 2016-03-01 20:19:09]

The patchset sent to the CQ was uploaded after l-g-t-m from eroman@chromium.org
Link to the patchset: https://codereview.chromium.org/1754813002/#ps130001
(title: "rebase tot")

[commit-bot: I haz the power, 2016-03-01 20:20:02]

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

[commit-bot: I haz the power, 2016-03-01 20:23:00]

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

[commit-bot: I haz the power, 2016-03-01 20:23:01]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ramant (doing other things), 2016-03-01 20:53:15]

The CQ bit was checked by rtenneti@chromium.org

[ramant (doing other things), 2016-03-01 20:53:16]

The patchset sent to the CQ was uploaded after l-g-t-m from eroman@chromium.org
Link to the patchset: https://codereview.chromium.org/1754813002/#ps150001
(title: "Rebase TOT")

[commit-bot: I haz the power, 2016-03-01 20:53:53]

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

[commit-bot: I haz the power, 2016-03-01 22:40:31]

Committed patchset #3 (id:150001)

[commit-bot: I haz the power, 2016-03-01 22:42:33]

Description was changed from

==========
Revert of QUIC - instrumentation for testing null QuicSpdyStream. (patchset #5
id:100001 of https://codereview.chromium.org/1688003005/ )

Reason for revert:
We got the call stack that causes the crash. Reverting this change. Will add the
check for nullptr and the unit tests. 

Original issue's description:
> QUIC - instrumentation for testing null QuicSpdyStream.
> 
> BUG=585591
> R=rch@chromium.org,eroman@chromium.org
> 
> Committed: https://crrev.com/100e39060a98afdc80f7a5140f6ba690746261b9
> Cr-Commit-Position: refs/heads/master@{#375804}

TBR=rch@chromium.org,eroman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=585591
==========

to

==========
Revert of QUIC - instrumentation for testing null QuicSpdyStream. (patchset #5
id:100001 of https://codereview.chromium.org/1688003005/ )

Reason for revert:
We got the call stack that causes the crash. Reverting this change. Will add the
check for nullptr and the unit tests.

Original issue's description:
> QUIC - instrumentation for testing null QuicSpdyStream.
>
> BUG=585591
> R=rch@chromium.org,eroman@chromium.org
>
> Committed: https://crrev.com/100e39060a98afdc80f7a5140f6ba690746261b9
> Cr-Commit-Position: refs/heads/master@{#375804}

TBR=rch@chromium.org,eroman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=585591

Committed: https://crrev.com/45e15292b538db8e1b60f4e0eeea1c0587e3c190
Cr-Commit-Position: refs/heads/master@{#378582}
==========

[commit-bot: I haz the power, 2016-03-01 22:42:34]

Patchset 3 (id:??) landed as
https://crrev.com/45e15292b538db8e1b60f4e0eeea1c0587e3c190
Cr-Commit-Position: refs/heads/master@{#378582}

[ramant (doing other things), 2016-03-02 21:58:20]

Description was changed from

==========
Revert of QUIC - instrumentation for testing null QuicSpdyStream. (patchset #5
id:100001 of https://codereview.chromium.org/1688003005/ )

Reason for revert:
We got the call stack that causes the crash. Reverting this change. Will add the
check for nullptr and the unit tests.

Original issue's description:
> QUIC - instrumentation for testing null QuicSpdyStream.
>
> BUG=585591
> R=rch@chromium.org,eroman@chromium.org
>
> Committed: https://crrev.com/100e39060a98afdc80f7a5140f6ba690746261b9
> Cr-Commit-Position: refs/heads/master@{#375804}

TBR=rch@chromium.org,eroman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=585591

Committed: https://crrev.com/45e15292b538db8e1b60f4e0eeea1c0587e3c190
Cr-Commit-Position: refs/heads/master@{#378582}
==========

to

==========
Revert of QUIC - instrumentation for testing null QuicSpdyStream. (patchset #5
id:100001 of https://codereview.chromium.org/1688003005/ )

Reason for revert:
We got the call stack that causes the crash. Reverting this change. Will add the
check for nullptr and the unit tests.

Original issue's description:
> QUIC - instrumentation for testing null QuicSpdyStream.
>
> BUG=585591
> R=rch@chromium.org,eroman@chromium.org
>
> Committed: https://crrev.com/100e39060a98afdc80f7a5140f6ba690746261b9
> Cr-Commit-Position: refs/heads/master@{#375804}

TBR=rch@chromium.org,eroman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=585591

Committed: https://crrev.com/45e15292b538db8e1b60f4e0eeea1c0587e3c190
Cr-Commit-Position: refs/heads/master@{#378582}
==========

[ramant (doing other things), 2016-03-02 21:58:20]

rtenneti@chromium.org changed reviewers:
+ ketakid@google.com