Bug 52817

Summary:

QtTestBrowser should have seperate menu for Settings

Product:

WebKit

Reporter:

mansi mithal <mansi.mithal>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, laszlo.gombos, tonikitoo

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Adds a new menu item and move the settings under it.	none

Description mansi mithal 2011-01-20 08:47:11 PST

QtTestBrowser should have seperate menu for "Settings" and move "Enable interrupting JS scripts" and "Enable js pop up windows" under the "Settings" menu

Comment 1 mansi mithal 2011-01-20 12:19:59 PST

Created attachment 79632 [details]
Adds a new menu item and move the settings under it.

Comment 2 Antonio Gomes 2011-01-20 13:20:57 PST

Comment on attachment 79632 [details]
Adds a new menu item and move the settings under it.

Why not moving Spatial Navigation, Frame flattening, etc at once?

Comment 3 Antonio Gomes 2011-01-20 13:22:07 PST

Also, you can use the "Depends on" and "Blocks" fields to set inter-dependencies between bugs.

Comment 4 WebKit Commit Bot 2011-01-20 14:48:26 PST

The commit-queue encountered the following flaky tests while processing attachment 79632 [details]:

http/tests/xmlhttprequest/cross-origin-authorization.html bug 52398 (author: ap@webkit.org)
The commit-queue is continuing to process your patch.

Comment 5 WebKit Commit Bot 2011-01-20 14:51:04 PST

Comment on attachment 79632 [details]
Adds a new menu item and move the settings under it.

Clearing flags on attachment: 79632

Committed r76286: <http://trac.webkit.org/changeset/76286>

Comment 6 WebKit Commit Bot 2011-01-20 14:51:09 PST

All reviewed patches have been landed.  Closing bug.