Bug 134289

Summary:

Rearrange conditions to find invalid protocol in NavigatorContentUtils::verifyProtocolHandlerScheme()

Product:

WebKit

Reporter:

Gyuyoung Kim <gyuyoung.kim>

Component:

WebCore Misc.

Assignee:

Gyuyoung Kim <gyuyoung.kim>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bunhere, cdumez, commit-queue, darin, gyuyoung.kim, sergio

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Gyuyoung Kim 2014-06-24 20:09:23 PDT

I think it would be good to do "early return" if there is invalid protocol.

Comment 1 Gyuyoung Kim 2014-06-24 20:13:41 PDT

Created attachment 233780 [details]
Patch

Comment 2 Gyuyoung Kim 2014-06-26 15:50:11 PDT

CC'ing Darin.

Comment 3 WebKit Commit Bot 2014-06-26 17:27:35 PDT

Comment on attachment 233780 [details]
Patch

Clearing flags on attachment: 233780

Committed r170511: <http://trac.webkit.org/changeset/170511>

Comment 4 WebKit Commit Bot 2014-06-26 17:27:41 PDT

All reviewed patches have been landed.  Closing bug.