Bug 81697 - Refactor notification tests requiring file:// permission
Summary: Refactor notification tests requiring file:// permission
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.7
: P2 Normal
Assignee: Jon Lee
URL:
Keywords: InRadar
Depends on: 95539 79492 95535 95537 95609
Blocks:
  Show dependency tree / graph
 
Reported: 2012-03-20 13:58 PDT by Jon Lee
Modified: 2012-08-31 14:18 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jon Lee 2012-03-20 13:58:32 PDT 
Because of 79704, some of the tests will not work by default because WebCore will short-circuit and not ask the client for permission. The tests need to be rewritten with this in mind, and possibly switch on universal access for file URLs for testing, or check for both 1 (default denied) and 2 (denied).
Comment 1 Radar WebKit Bug Importer 2012-03-20 13:59:05 PDT 
<rdar://problem/11084702>
Comment 2 Kristóf Kosztyó 2012-03-21 03:18:39 PDT 
We skipped these tests on Qt in r111519 to make the bot green.
http://trac.webkit.org/changeset/111519
Comment 3 Jon Lee 2012-08-30 13:10:47 PDT 
To refactor the tests, I will basically be moving all of the tests from fast/notifications to http/tests/notifications. But the state of the tests seem to be pretty bad. Some of them are disabled, and I'm unsure why. And some tests I think should be deprecated.

This bug will be the master bug tracking the effort.