Bug 95972

Summary: REGRESSION(r127712): It broke http/tests/xmlhttprequest/basic-auth.html on JSC platforms
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Critical CC: abarth, haraken, mitz, ossy, thorton
Priority: P2 Keywords: LayoutTestFailure, Regression
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 79668, 95903    

Csaba Osztrogonác
Reported 2012-09-06 04:15:09 PDT
--- /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/http/tests/xmlhttprequest/basic-auth-expected.txt +++ /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/http/tests/xmlhttprequest/basic-auth-actual.txt @@ -1,16 +1,15 @@ +FAIL: Timed out waiting for notifyDone to be called Tests for XMLHttpRequest authentication. sync: User: sync, password: 123. sync2: User: sync2, password: 123. sync3: User: sync3, password: 123. sync4: User: sync4, password: 123. -sync5: User: sync5, password: 123. -sync6: User: sync6, password: 123. -sync7: User: sync7, password: 123. +sync5: +sync6: +sync7: async: User: async, password: 123. async2: User: async2, password: 123. async3: User: async3, password: 123. async4: User: async4, password: 123. -async5: User: async5, password: 123. -async6: User: async6, password: 123. -async7: User: async7, password: 123. +async5 failed with unexpected status: 401
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2012-09-06 04:22:47 PDT
I skipped it on Qt to paint the bot green - https://trac.webkit.org/changeset/127728 Please unskip it with the proper fix.
mitz
Comment 2 2012-09-06 10:00:27 PDT
The JavaScriptCore bindings convert explicit undefined values for the user and password parameters into the string "undefined".
mitz
Comment 3 2012-09-06 10:11:23 PDT
Fixed in <http://trac.webkit.org/r127751> by reverting r127712.
Adam Barth
Comment 4 2012-09-06 10:39:42 PDT
Interesting. Sounds like we need to use TreatNullAs and TreatUndefinedAs.
Note You need to log in before you can comment on or make changes to this bug.