Bug 217409

Summary: [webkitcorepy] Handle case where arguments to memoized function are lists
Product: WebKit Reporter: Jonathan Bedard <jbedard>
Component: Tools / TestsAssignee: Jonathan Bedard <jbedard>
Status: RESOLVED WONTFIX    
Severity: Normal CC: darin, dewei_zhu, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=216404
Attachments:
Description Flags
Patch darin: review+, jbedard: commit-queue+

Description Jonathan Bedard 2020-10-06 15:28:17 PDT
At the moment, we don't handle lists being passed to memoized functions.
Comment 1 Radar WebKit Bug Importer 2020-10-06 15:28:31 PDT
<rdar://problem/70020109>
Comment 2 Jonathan Bedard 2020-10-06 16:56:30 PDT
Created attachment 410714 [details]
Patch
Comment 3 Jonathan Bedard 2020-10-07 11:27:12 PDT
Discussed this with Dean and Dewei, this case is a bit of a weird one because it handles what happens when someone passes a mutable argument to a memoized function. We're going to mark this as invalid and move on, the code change does what it says it does, it's just not obvious that behavior is more correct than the exception Python will raise.