Comment #3 on issue 18665 by [email protected]: block-test{-no-port}  
test results differ between windows and mac/linux
http://code.google.com/p/chromium/issues/detail?id=18665

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=22686

------------------------------------------------------------------------
r22686 | [email protected] | 2009-08-06 16:00:35 -0700 (Thu, 06 Aug 2009)  
| 13 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/platform/chromium-win/LayoutTests/security/block-test-expected.txt?r1=22686&r2=22685
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/layout_tests/test_expectations.txt?r1=22686&r2=22685
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_shell_request_context.cc?r1=22686&r2=22685
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_shell_switches.cc?r1=22686&r2=22685
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_shell_switches.h?r1=22686&r2=22685
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_webview_delegate.cc?r1=22686&r2=22685

Enable new FTP for Windows TestShell so that the results of block-test.html  
are
the same across platforms.  WTC tells me that it should be ready to go, and  
it
anyways is only exercised by these two security/block-test*.html tests.

Also, add back the fix from r22651.

Extend that fix a bit so that when we do not have a special mapping, we  
just spit out the existing error code value.  This way, if we get an  
unexpected result, the output will tell us what the original error code  
was.  This change doesn't impact any layout tests.

R=dglazkov,wtc
BUG=18665
TEST=none

Review URL: http://codereview.chromium.org/165091
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to