RESOLVED FIXED 27407
[V8] Revert incomplete patch 
https://bugs.webkit.org/show_bug.cgi?id=27407
Summary [V8] Revert incomplete patch
Jeremy Orlow
Reported 2009-07-18 00:29:53 PDT
Revert https://bugs.webkit.org/show_bug.cgi?id=27383 It's missing a file and we want the canaries to be green over the weekend.
Attachments
v1 (40.35 KB, patch)
2009-07-18 00:31 PDT, Jeremy Orlow 		no flags
DetailsFormatted DiffDiff
now with changelog! (42.43 KB, patch)
2009-07-18 00:36 PDT, Jeremy Orlow 		abarth: review+
DetailsFormatted DiffDiff
patch, with missing file added (47.45 KB, patch)
2009-07-20 09:39 PDT, Jens Alfke 		levin: review-
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Jeremy Orlow
Comment 1 2009-07-18 00:31:20 PDT
Created attachment 33012 [details] v1 Revert Jens' patch.
Jeremy Orlow
Comment 2 2009-07-18 00:36:08 PDT
Created attachment 33013 [details] now with changelog! add a change log
Adam Barth
Comment 3 2009-07-18 00:38:26 PDT
Comment on attachment 33013 [details] now with changelog! jorlow certifies that this is a revert of a previous patch. I apply the rubber stamp.
David Levin
Comment 4 2009-07-19 20:38:13 PDT
Committed as http://trac.webkit.org/changeset/46087 Please resolve the bug after committing the patch :)
Jens Alfke
Comment 5 2009-07-20 09:39:42 PDT
Created attachment 33090 [details] patch, with missing file added Added the missing WebCore/bindings/v8/custom/V8DataGridColumnListCustom.cpp; sorry about that. This patch is up to date with the current r46125.
Jeremy Orlow
Comment 6 2009-07-20 09:45:49 PDT
Comment on attachment 33090 [details] patch, with missing file added I don't have review rights. I think it's best for everyone if you leave off the requestee since really any reviewer could r+ this for you.
David Levin
Comment 7 2009-07-20 09:55:25 PDT
Comment on attachment 33090 [details] patch, with missing file added The patch is about fixing bug 27383, so it should be added to that bug. (Plus you'd want to re-open any bug that has a new patch on it and clear the r+ from any old patches.) *Also* in WebCore/bindings/v8/V8DataGridDataSource.cpp, this is missing a "//": #endif ENABLE(DATAGRID)
Note You need to log in before you can comment on or make changes to this bug.