Bug 59406
| Summary: | loader fix broke a basic auth test in chromium | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Dirk Pranke <dpranke> |
| Component: | Page Loading | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | ap, japhet, mjs, schenney |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | PC | ||
| OS: | OS X 10.5 | ||
Dirk Pranke
The fix in bug 59241 (http://trac.webkit.org/changeset/84742) broke one of the automated UI tests in Chromium that handles navigating back and forth between pages that use basic authentication. Th test expects to get a basic auth challenge, and instead the resource load appears to be succeeding.
The test in question is here:
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/login/login_prompt_uitest.cc?annotate=74158
That unit test is already marked as flaky on Windows, so maybe it is badly written and the loader fix is correct? I will disable that test for now but we need to figure out what to do about it.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Alexey Proskuryakov
The change only affects canceled main resource loads. Is the test intentionally canceling any?