[TS-4008] Cleanup SPDY ClientSessionSession / SpdyRequest Allocate and Deallocate code

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 6.1.0
• Component/s: Core, SPDY
• Assignee: Brian Geffon
• Reporter: Brian Geffon
• Created: 10/Nov/15 06:40
• Updated: 10/Nov/15 07:26
• Resolved: 10/Nov/15 07:26

Description

To make the SPDY classes SpdyClientSession and SpdyRequest conform with the rest of the code base's pattern of alloc()/destroy() we want to clean up that code in the following ways:

• Removed custom dtors from SpdyClientSession and SpdyRequest
• Implemented a uniform interface destroy() to both and refactored calls to clear() followed by ClassAllocator::free() with just destroy() to perform both.
• Implemented a static function alloc() for both classes that will use the designated ClassAllocator to alloc() the object.

This was also discussed in the pull request : https://github.com/apache/trafficserver/pull/323