RESOLVED FIXED 73631
[chromium] Add layout test support for autosize code in FrameView. 
https://bugs.webkit.org/show_bug.cgi?id=73631
Summary [chromium] Add layout test support for autosize code in FrameView.
David Levin
Reported 2011-12-01 23:45:11 PST
Only chromium because that is the only place where I'm exposing a WebKit api for this.
Attachments
Patch (12.83 KB, patch)
2012-01-06 13:49 PST, David Levin 		no flags
DetailsFormatted DiffDiff
Patch for landing (12.88 KB, patch)
2012-01-06 14:23 PST, David Levin 		no flags
DetailsFormatted DiffDiff
Patch for landing (12.90 KB, patch)
2012-01-07 22:59 PST, David Levin 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
David Levin
Comment 1 2011-12-01 23:49:31 PST
And add tests using that support of course.
David Levin
Comment 2 2012-01-06 13:49:18 PST
Created attachment 121485 [details] Patch
Dmitry Titov
Comment 3 2012-01-06 14:03:25 PST
Comment on attachment 121485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=121485&action=review r+ with a nit. If there will be more tests in this dir, lets share things like shouldBe() between them. > Tools/DumpRenderTree/chromium/LayoutTestController.cpp:1107 > + return; result->set(false); ?
David Levin
Comment 4 2012-01-06 14:23:51 PST
Created attachment 121496 [details] Patch for landing
WebKit Review Bot
Comment 5 2012-01-06 18:29:08 PST
Comment on attachment 121496 [details] Patch for landing Rejecting attachment 121496 [details] from commit-queue. New failing tests: fast/autoresize/basic.html Full output: http://queues.webkit.org/results/11171171
David Levin
Comment 6 2012-01-07 22:59:36 PST
Created attachment 121568 [details] Patch for landing
WebKit Review Bot
Comment 7 2012-01-08 00:06:54 PST
Comment on attachment 121568 [details] Patch for landing Clearing flags on attachment: 121568 Committed r104398: <http://trac.webkit.org/changeset/104398>
WebKit Review Bot
Comment 8 2012-01-08 00:06:59 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.