RESOLVED FIXED133615
ScriptExecutionContext::Task parameters should always be rvalue references 
https://bugs.webkit.org/show_bug.cgi?id=133615
Summary ScriptExecutionContext::Task parameters should always be rvalue references
Zan Dobersek
Reported 2014-06-07 22:49:22 PDT
ScriptExecutionContext::Task parameters should always be rvalue references
Attachments
Patch (14.21 KB, patch)
2014-06-07 22:58 PDT, Zan Dobersek 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Zan Dobersek
Comment 1 2014-06-07 22:58:47 PDT
Created attachment 232679 [details] Patch
Darin Adler
Comment 2 2014-06-17 13:43:12 PDT
Comment on attachment 232679 [details] Patch Wait, I am not sure this is a good idea. If the object really is move-only then it seems strange to change it to &&.
Darin Adler
Comment 3 2014-06-17 14:06:53 PDT
Anders, what do you think of this from a coding style point of view?
Anders Carlsson
Comment 4 2014-06-17 14:24:15 PDT
(In reply to comment #3) > Anders, what do you think of this from a coding style point of view? If it's move only it doesn't matter. I don't think it should be an rvalue reference in that case.
Zan Dobersek
Comment 5 2014-06-18 10:54:46 PDT
Committed r170106: <http://trac.webkit.org/changeset/170106>
Zan Dobersek
Comment 6 2014-06-18 11:05:14 PDT
Sorry, I missed the comments before landing. I'll revert these and other ScriptExecutionContext::Task&& parameters in one go, in a different bug.
Note You need to log in before you can comment on or make changes to this bug.