Bug 159710

Summary: Test WTF.StringViewIterators is crashing since r203119
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, lforschler, mmaxfield
Priority: P2    
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=159609
Attachments:
Description Flags
Patch darin: review+

Description Carlos Garcia Campos 2016-07-13 03:53:35 PDT
I think there are two problems in the test case added in r203119. First is that it's testing StringView::CodeUnits::Iterator instead of StringView::CodePoints::Iterator, and second is that StringView::codePoints() returns a temporary, so the iterator created from the temporary ends up pointing to a const reference to the string view member of the deleted CodePoints object.
Comment 1 Carlos Garcia Campos 2016-07-13 03:55:05 PDT
Created attachment 283508 [details]
Patch
Comment 2 Myles C. Maxfield 2016-07-13 12:16:59 PDT
Wow... whoops. Thanks so much for fixing this.
Comment 3 Carlos Garcia Campos 2016-07-13 23:18:31 PDT
Committed r203214: <http://trac.webkit.org/changeset/203214>