Bug 120013

Summary: Tighten up logic in HTMLTableRowsCollection
Product: WebKit Reporter: Darin Adler <darin>
Component: TablesAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, esprehn+autocc
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Description Darin Adler 2013-08-19 10:07:15 PDT
Tighten up logic in HTMLTableRowsCollection
Comment 1 Darin Adler 2013-08-19 10:13:13 PDT
Created attachment 209099 [details]
Patch
Comment 2 Andy Estes 2013-08-19 11:02:22 PDT
Comment on attachment 209099 [details]
Patch

My preference would be to only have one definition of assertRowIsInTable() and just mark row as unused in the ASSERT_DISABLED case, but r=me either way.
Comment 3 WebKit Commit Bot 2013-08-19 12:10:37 PDT
Comment on attachment 209099 [details]
Patch

Clearing flags on attachment: 209099

Committed r154288: <http://trac.webkit.org/changeset/154288>
Comment 4 WebKit Commit Bot 2013-08-19 12:10:39 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Brent Fulgham 2013-08-21 09:34:39 PDT
Reopening to attach new patch.
Comment 6 Brent Fulgham 2013-08-21 09:34:40 PDT
Created attachment 209274 [details]
Patch
Comment 7 Brent Fulgham 2013-08-21 09:36:17 PDT
Accidentally updated this bug, when I meant to change Bug 120113.

Sorry for the noise.