Modernize HTMLConstructionSite.
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.