Summary: | Parse HTMLOLElement.start as per the HTML spec | ||||||
---|---|---|---|---|---|---|---|
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, rniwa, sam | ||||
Priority: | P2 | Keywords: | WebExposed | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Chris Dumez
2016-02-28 15:00:41 PST
Created attachment 272467 [details]
Patch
Comment on attachment 272467 [details]
Patch
r=me
Comment on attachment 272467 [details] Patch Clearing flags on attachment: 272467 Committed r197307: <http://trac.webkit.org/changeset/197307> All reviewed patches have been landed. Closing bug. What in the world is that 0xBADBEEF thing about? (In reply to comment #5) > What in the world is that 0xBADBEEF thing about? We only read m_start if m_hasExplicitStart is true. If m_hasExplicitStart is false, we set m_start to 0xBADBEEF to make it clear its value is invalid. Honestly, I feel this should be refactored to use WTF::Optional<>. I'll probably do this in a follow-up. |