Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all of its implementations (and related code) can be deleted.

This CL lands server change 135920024 by birenroy.

BUG=488484
Committed: https://crrev.com/73d2da9ae04cb72258a4c8cbac7a90a0677c1006
Cr-Commit-Position: refs/heads/master@{#426875}

[yasong, 2016-10-20 21:45:47]

The CQ bit was checked by yasong@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-20 21:46:49]

Dry run: 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-10-20 23:23:47]

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

[commit-bot: I haz the power, 2016-10-20 23:23:47]

Dry run: This issue passed the CQ dry run.

[yasong, 2016-10-20 23:27:26]

Description was changed from

==========
Removes SpdyHeaderBlock::GetHeader(). No functional change. Not protected.

Equivalent functionality is provided by SpdyHeaderBlock::find().

This CL lands server change 135849112 by birenroy.

BUG=488484
==========

to

==========
Removes SpdyHeaderBlock::GetHeader(). No functional change. Not protected.

Equivalent functionality is provided by SpdyHeaderBlock::find().

This CL lands server change 135849112 by birenroy.

BUG=488484
==========

[yasong, 2016-10-20 23:27:26]

yasong@chromium.org changed reviewers:
+ bnc@chromium.org

[yasong, 2016-10-20 23:27:44]

On 2016/10/20 23:27:26, yasong wrote:
> mailto:yasong@chromium.org changed reviewers:
> + mailto:bnc@chromium.org

Ready for review. Thanks!

[yasong, 2016-10-21 00:28:20]

Description was changed from

==========
Removes SpdyHeaderBlock::GetHeader(). No functional change. Not protected.

Equivalent functionality is provided by SpdyHeaderBlock::find().

This CL lands server change 135849112 by birenroy.

BUG=488484
==========

to

==========
SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all
of its implementations (and related code) can be deleted.
    
This CL lands server change 135920024 by birenroy.
    
BUG=488484
==========

[Bence, 2016-10-21 00:34:51]

On 2016/10/20 23:27:44, yasong wrote:
> On 2016/10/20 23:27:26, yasong wrote:
> > mailto:yasong@chromium.org changed reviewers:
> > + mailto:bnc@chromium.org
> 
> Ready for review. Thanks!

If I'm not mistaken, this CL seems to include two server side changes: 135920024
(not mentioned in the CL description at all, please fix), and 135849112 (being
merged separately at https://crrev.com/2431413004, please rebase this CL once
that one lands).  Thank you.

[yasong, 2016-10-21 16:12:18]

The CQ bit was checked by yasong@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-21 16:12:36]

Dry run: 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-10-21 17:23:45]

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

[commit-bot: I haz the power, 2016-10-21 17:23:46]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-10-21 19:54:32]

Description was changed from

==========
SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all
of its implementations (and related code) can be deleted.
    
This CL lands server change 135920024 by birenroy.
    
BUG=488484
==========

to

==========
Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all
of its implementations (and related code) can be deleted.
    
This CL lands server change 135920024 by birenroy.
    
BUG=488484
==========

[Bence, 2016-10-21 20:00:27]

LGTM.

[yasong1, 2016-10-21 20:02:05]

The CQ bit was checked by yasong@google.com

[commit-bot: I haz the power, 2016-10-21 20:02:40]

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-10-21 20:26:02]

Description was changed from

==========
Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all
of its implementations (and related code) can be deleted.
    
This CL lands server change 135920024 by birenroy.
    
BUG=488484
==========

to

==========
Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all
of its implementations (and related code) can be deleted.
    
This CL lands server change 135920024 by birenroy.
    
BUG=488484
==========

[commit-bot: I haz the power, 2016-10-21 20:26:03]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-21 20:27:38]

Description was changed from

==========
Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all
of its implementations (and related code) can be deleted.
    
This CL lands server change 135920024 by birenroy.
    
BUG=488484
==========

to

==========
Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

SpdyFramerVisitorInterface::OnControlFrameHeaderData() is never called, so all
of its implementations (and related code) can be deleted.

This CL lands server change 135920024 by birenroy.

BUG=488484

Committed: https://crrev.com/73d2da9ae04cb72258a4c8cbac7a90a0677c1006
Cr-Commit-Position: refs/heads/master@{#426875}
==========

[commit-bot: I haz the power, 2016-10-21 20:27:39]

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