Bug 81580

Summary: webkitpy: allow instance data in TestFailure objects
Product: WebKit Reporter: Dirk Pranke <dpranke>
Component: New BugsAssignee: Dirk Pranke <dpranke>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, ojan, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 71380    
Attachments:
Description Flags
Patch tony: review+

Description Dirk Pranke 2012-03-19 16:08:04 PDT
webkitpy: allow instance data in TestFailure objects
Comment 1 Dirk Pranke 2012-03-19 16:08:52 PDT
Created attachment 132699 [details]
Patch
Comment 2 Eric Seidel (no email) 2012-03-19 16:22:36 PDT
Comment on attachment 132699 [details]
Patch

I feel like I attempted this change before and someone shot it down.  Don't these things get serialized/de-serialized as part of being passed around?  I'm fine with this change, but Ojan may have opinions?
Comment 3 Tony Chang 2012-03-19 16:34:06 PDT
Comment on attachment 132699 [details]
Patch

Looks like the reftest failure objects are already instances.  Should be fine as long as we don't associate anything with the common failure types (which we create lots of instances of).
Comment 4 Dirk Pranke 2012-03-19 16:45:00 PDT
(In reply to comment #2)
> (From update of attachment 132699 [details])
> I feel like I attempted this change before and someone shot it down.  Don't these things get serialized/de-serialized as part of being passed around?  I'm fine with this change, but Ojan may have opinions?

I may have shot it down before :) They do get serialized, but since we're just adding strings and ints, this is safe and although I didn't like the idea of instance data before, I don't quite recall why I didn't, and it's clearly useful now.
Comment 5 Dirk Pranke 2012-03-19 18:21:26 PDT
Committed r111287: <http://trac.webkit.org/changeset/111287>