Bug 117982

Summary:

[BlackBerry] Handle testRunner.setCustomPolicyDelegate()

Product:

WebKit

Reporter:

Carlos Garcia Campos <cgarcia>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

rwlbuis

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	rwlbuis: review+

Carlos Garcia Campos

Reported 2013-06-25 05:39:33 PDT

It's currently implemented, but the permissive flag is ignored.

Attachments
Patch (6.12 KB, patch) 2013-06-25 05:47 PDT, Carlos Garcia Campos	rwlbuis: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Carlos Garcia Campos

Comment 1 2013-06-25 05:47:10 PDT

Created attachment 205386 [details] Patch Fixes test fast/loader/onload-policy-ignore-for-frame.html

Rob Buis

Comment 2 2013-06-26 12:43:32 PDT

Comment on attachment 205386 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205386&action=review LGTM. > Source/WebKit/blackberry/ChangeLog:8 > + Take custom cpolicy into account when deciding the policy for I assume you mean 'policy' here instead of 'cpolicy'.

Carlos Garcia Campos

Comment 3 2013-06-26 23:57:22 PDT

(In reply to comment #2) > (From update of attachment 205386 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=205386&action=review > > LGTM. > > > Source/WebKit/blackberry/ChangeLog:8 > > + Take custom cpolicy into account when deciding the policy for > > I assume you mean 'policy' here instead of 'cpolicy'. Oops, yes, it's a typo, will fix it before landing.

Carlos Garcia Campos

Comment 4 2013-06-27 00:00:47 PDT

Committed r152081: <http://trac.webkit.org/changeset/152081>

Note You need to log in before you can comment on or make changes to this bug.