Bug 175190

Summary:

Cleanup: simplify WebSockets code for RuntimeEnabledFeatures

Product:

WebKit

Reporter:

BJ Burg <bburg>

Component:

WebCore Misc.

Assignee:

BJ Burg <bburg>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bburg, commit-queue, joepeck, sam, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

167044

Bug Blocks:

Attachments:

Description	Flags
Patch	none

Description BJ Burg 2017-08-04 09:16:22 PDT

Comment 1 BJ Burg 2017-08-04 14:59:30 PDT

Created attachment 317297 [details]
Patch

Comment 2 Joseph Pecoraro 2017-08-05 16:46:45 PDT

Comment on attachment 317297 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=317297&action=review

> Source/WebCore/Modules/websockets/WebSocket.idl:-38
> -    EnabledAtRuntime=WebSocket,

It's not accurate to say it wasn't used for anything when it was used for this! But I guess we don't expect anyone to toggle it off.

Comment 3 WebKit Commit Bot 2017-08-07 09:32:28 PDT

Comment on attachment 317297 [details]
Patch

Clearing flags on attachment: 317297

Committed r220337: <http://trac.webkit.org/changeset/220337>

Comment 4 WebKit Commit Bot 2017-08-07 09:32:29 PDT

All reviewed patches have been landed.  Closing bug.

Comment 5 Radar WebKit Bug Importer 2017-08-07 09:32:47 PDT

<rdar://problem/33754436>