Rename websocketClientFlipperConnection
Summary: At the moment Flipper spawns a WebSocket server which can be used to debug Web applications in the Browser i.e. Kite. This diff renames websocketClientFlipperConnection to BrowserClientFlipperConnection. Renamed to make it clearer what type of connection it really is. WebSocket connections will no longer have the same meaning once RSocket is removed and WebSockets are used instead. Reviewed By: jknoxville Differential Revision: D29937656 fbshipit-source-id: 99d55fcfe256865add1ba3aada2c23f935f5cabe
This commit is contained in:
committed by
Facebook GitHub Bot
parent
dcb4b2fb96
commit
1f1f41c81e
@@ -38,7 +38,7 @@ import {buildClientId} from '../utils/clientUtils';
|
||||
import {Single} from 'rsocket-flowable';
|
||||
import WebSocket from 'ws';
|
||||
import JSDevice from '../devices/JSDevice';
|
||||
import {WebsocketClientFlipperConnection} from '../utils/js-client-server-utils/websocketClientFlipperConnection';
|
||||
import {BrowserClientFlipperConnection} from '../utils/js-client-server-utils/BrowserClientFlipperConnection';
|
||||
import querystring from 'querystring';
|
||||
import {IncomingMessage} from 'http';
|
||||
import ws from 'ws';
|
||||
@@ -217,7 +217,7 @@ class ServerController extends EventEmitter {
|
||||
const plugins = message.plugins;
|
||||
let resolvedClient: Client | null = null;
|
||||
const client = this.addConnection(
|
||||
new WebsocketClientFlipperConnection(ws, app, plugins),
|
||||
new BrowserClientFlipperConnection(ws, app, plugins),
|
||||
{
|
||||
app,
|
||||
os: 'JSWebApp',
|
||||
|
||||
@@ -15,7 +15,7 @@ import {
|
||||
ResponseType,
|
||||
} from '../../comms/ClientConnection';
|
||||
|
||||
export class WebsocketClientFlipperConnection implements ClientConnection {
|
||||
export class BrowserClientFlipperConnection implements ClientConnection {
|
||||
websocket: WebSocket;
|
||||
connStatusSubscribers: Set<ConnectionStatusChange> = new Set();
|
||||
connStatus: ConnectionStatus;
|
||||
Reference in New Issue
Block a user