Summary: | Use WTF::Optional for HTMLOListElement::m_start | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, darin, esprehn+autocc, gyuyoung.kim, kling, rniwa, sam | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Chris Dumez
2016-02-29 10:37:41 PST
Created attachment 272500 [details]
Patch
Created attachment 272501 [details]
Patch
Comment on attachment 272501 [details]
Patch
r=me
Comment on attachment 272501 [details] Patch Clearing flags on attachment: 272501 Committed r197364: <http://trac.webkit.org/changeset/197364> All reviewed patches have been landed. Closing bug. Now we just have to change those parseHTML functions to return an Optional<> rather than returning a boolean and using an out argument for the parsed values. (In reply to comment #6) > Now we just have to change those parseHTML functions to return an Optional<> > rather than returning a boolean and using an out argument for the parsed > values. Ah, that sounds good indeed :) |