Summary: | Modernize HTMLConstructionSite | ||||||||
---|---|---|---|---|---|---|---|---|---|
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, kangil.han, koivisto, rniwa | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Chris Dumez
2016-05-02 14:14:02 PDT
Created attachment 277933 [details]
Patch
Comment on attachment 277933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=277933&action=review > Source/WebCore/html/parser/HTMLConstructionSite.cpp:723 > + return firstUnopenElementIndex < m_activeFormattingElements.size() ? firstUnopenElementIndex : Optional<unsigned>(Nullopt); Doesn't just Nullopt work? > Source/WebCore/html/parser/HTMLFormattingElementList.h:53 > + { } Don't we like these on different lines in this case? > Source/WebCore/html/parser/HTMLFormattingElementList.h:56 > + { } ditto > Source/WebCore/html/parser/HTMLFormattingElementList.h:83 > + { } ditto Comment on attachment 277933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=277933&action=review >> Source/WebCore/html/parser/HTMLConstructionSite.cpp:723 >> + return firstUnopenElementIndex < m_activeFormattingElements.size() ? firstUnopenElementIndex : Optional<unsigned>(Nullopt); > > Doesn't just Nullopt work? No, it did not build. Created attachment 277940 [details]
Patch
Comment on attachment 277940 [details] Patch Clearing flags on attachment: 277940 Committed r200340: <http://trac.webkit.org/changeset/200340> All reviewed patches have been landed. Closing bug. |