Rebaseline some layout-tests which changed in upstrea webkit r46412....

Rebaseline some layout-tests which changed in upstream webkit r46412.

BUG=http://crbug.com/17326


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=21245

[rniwa, 2009-07-21 21:46:42]

Thank you for rebaselining it.
I think we figured out other regression about 19087.html & 19653-1.html.  It
turned out that there was some hidden bug in Position.h.  Will be submitting
a patch soon.
Ryosuke

On Tue, Jul 21, 2009 at 2:43 PM, <eroman@chromium.org> wrote:

> Reviewers: rniwa,
>
> Description:
> Rebaseline some layout-tests which changed in upstrea webkit r46412.
>
> BUG=http://crbug.com/17326
>
>
> Please review this at http://codereview.chromium.org/159159
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  MM
> 
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/editing/execCommand/indent-selection-expected.checksum
>  M
>
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/editing/execCommand/indent-selection-expected.png
>  M
>
webkit/data/layout_tests/platform/chromium-win/LayoutTests/editing/execCommand/4641880-2-expected.txt
>  M
>
webkit/data/layout_tests/platform/chromium-win/LayoutTests/editing/execCommand/4916541-expected.txt
>  M
>
webkit/data/layout_tests/platform/chromium-win/LayoutTests/editing/execCommand/5138441-expected.txt
>  M
>
webkit/data/layout_tests/platform/chromium-win/LayoutTests/editing/execCommand/5481523-expected.txt
>  MM
> 
webkit/data/layout_tests/platform/chromium-win/LayoutTests/editing/execCommand/indent-selection-expected.checksum
>  M
>
webkit/data/layout_tests/platform/chromium-win/LayoutTests/editing/execCommand/indent-selection-expected.png
>  M
>
webkit/data/layout_tests/platform/chromium-win/LayoutTests/editing/execCommand/indent-selection-expected.txt
>  M     webkit/tools/layout_tests/test_expectations.txt
>
>
>

[eroman, 2009-07-21 21:48:41]

So should I hold off on the rebaseline until this happens?

[rniwa, 2009-07-21 22:04:02]

On Tue, Jul 21, 2009 at 2:48 PM, <eroman@chromium.org> wrote:

> So should I hold off on the rebaseline until this happens?
>
>
> http://codereview.chromium.org/159159
>

You can still rebaseline tests.  My new patch only fixes REGRESSION
(r46142): editing/execCommand/19087.html & editing/execCommand/19653-1.html.
It doesn't change any other test results I rebaselined. Thanks for follow up
though.
Best regards,
Ryosuke

[rniwa, 2009-07-21 23:11:20]

Hi Eric:
I just landed my patch for the regression.  I should make the tree happy
soon.

Ryosuke

On Tue, Jul 21, 2009 at 3:03 PM, Ryosuke Niwa <rniwa@google.com> wrote:

> On Tue, Jul 21, 2009 at 2:48 PM, <eroman@chromium.org> wrote:
>
>> So should I hold off on the rebaseline until this happens?
>>
>>
>> http://codereview.chromium.org/159159
>>
>
> You can still rebaseline tests.  My new patch only fixes REGRESSION
> (r46142): editing/execCommand/19087.html & editing/execCommand/19653-1.html.
> It doesn't change any other test results I rebaselined. Thanks for follow up
> though.
> Best regards,
> Ryosuke
>
>