for consistency
Created attachment 153690 [details] Patch
Comment on attachment 153690 [details] Patch Attachment 153690 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/13311857
Created attachment 153691 [details] Patch 2 Qt fix
Comment on attachment 153691 [details] Patch 2 View in context: https://bugs.webkit.org/attachment.cgi?id=153691&action=review > Source/WebCore/html/shadow/MeterShadowElement.h:87 > +#endif > #endif If there are two or more #endif s, we want to put '#endif // METER_ELEMENT' and '#endif // MeterShadowElement_h'. > Source/WebCore/html/shadow/ProgressShadowElement.h:88 > +#endif > #endif Ditto. > LayoutTests/fast/dom/wrapper-classes.html:281 > + // Conditional elements, VIDEO, PROGRESS_ELEMENT, RUBY, DATALIST_ELEMENT, METER_ELEMENT. Nit: DATALIST => DATALIST_ELEMENT is intended? Looks good though.
Committed r123300: <http://trac.webkit.org/changeset/123300>
Comment on attachment 153691 [details] Patch 2 View in context: https://bugs.webkit.org/attachment.cgi?id=153691&action=review >> Source/WebCore/html/shadow/MeterShadowElement.h:87 >> #endif > > If there are two or more #endif s, we want to put '#endif // METER_ELEMENT' and '#endif // MeterShadowElement_h'. Done. >> Source/WebCore/html/shadow/ProgressShadowElement.h:88 >> #endif > > Ditto. Done. >> LayoutTests/fast/dom/wrapper-classes.html:281 >> + // Conditional elements, VIDEO, PROGRESS_ELEMENT, RUBY, DATALIST_ELEMENT, METER_ELEMENT. > > Nit: DATALIST => DATALIST_ELEMENT is intended? Looks good though. Yes. We shouldn't add unrelated changes basically, however I think it's ok in this case because I'm modifying this line.