There's a FIXME in RegExp.h stating that it shouldn't be necessary to store the original flags string. Well, actually it isn't; RegExp.prototype.toString already uses the flag bits to reconstruct the string.
Created attachment 46050 [details] proposed patch
style-queue ran check-webkit-style on attachment 46050 [details] without any errors.
Comment on attachment 46050 [details] proposed patch Clearing flags on attachment: 46050 Committed r52983: <http://trac.webkit.org/changeset/52983>
All reviewed patches have been landed. Closing bug.
Reverted r52983 for reason: Broke 59 JavaScriptCore tests. I don't think Kent knew about run-javascriptcore-tests. Sadly neither does the commit-bot, yet. Committed r52987: <http://trac.webkit.org/changeset/52987>
I think my rollout may have been wrong. :( I think the bots are just flakey. They reported more failures after the first failures which seemed to be correlated to this patch. I'll wait for them to calm down and then roll this back in if necessary. Lesson: You can't trust the bots.
I will have the commit-queue land this again one bug 33376 is landed.
Comment on attachment 46050 [details] proposed patch Sorry for the troubles. The commit-queue will re-land this shortly (and will re-confirm that it does not break run-javascriptcore-tests before landing).
Comment on attachment 46050 [details] proposed patch Clearing flags on attachment: 46050 Committed r53032: <http://trac.webkit.org/changeset/53032>