[webkit-reviews] review requested: [Bug 3905] Missing </title> makes page blank : [Attachment 6773] Patch v6

[bugzilla-request-daemon at opendarwin.org]

David Kilzer (ddkilzer) <ddkilzer at kilzer.net> has asked  for review:
Bug 3905: Missing </title> makes page blank
http://bugzilla.opendarwin.org/show_bug.cgi?id=3905

Attachment 6773: Patch v6
http://bugzilla.opendarwin.org/attachment.cgi?id=6773&action=edit

------- Additional Comments from David Kilzer (ddkilzer) <ddkilzer at kilzer.net>
Updates from Patch v5:

- Removed verbose comments from htmlparser.cpp patch.
- Added copy constructor and assignment operator to SegmentedString class.

Notes:

- In the SegmentedString class, I can't find any code where "m_currentChar =
&m_pushedChar2", so I'm not sure the middle else-if logic is really necessary
in the copy constructor and assignment operator.
- I did not put the operator=() prototype next to the other operator prototypes
in SegmentedString.h.  I thought it best to leave it by the copy constructor. 
Let me know if this needs to be moved.
- All layout tests passed with this new patch.