chore: cleanup code that is not used by cli anymore (#5005)

Author: dgozman

src/client/frame.ts

@@ -464,12 +464,6 @@ export class Frame extends ChannelOwner<channels.FrameChannel, channels.FrameIni
       return (await this._channel.title()).value;
     });
   }
-
-  // TODO: remove once playwright-cli does not use this one anymore.
-  async _extendInjectedScript<Arg>(source: string, arg?: Arg): Promise<JSHandle> {
-    const result = await this._channel.extendInjectedScript({ source, arg: serializeArgument(arg) });
-    return JSHandle.from(result.handle);
-  }
 }
 
 export function verifyLoadState(name: string, waitUntil: LifecycleEvent): LifecycleEvent {

src/dispatchers/frameDispatcher.ts

@@ -233,8 +233,4 @@ export class FrameDispatcher extends Dispatcher<Frame, channels.FrameInitializer
   async title(): Promise<channels.FrameTitleResult> {
     return { value: await this._frame.title() };
   }
-
-  async extendInjectedScript(params: channels.FrameExtendInjectedScriptParams): Promise<channels.FrameExtendInjectedScriptResult> {
-    return { handle: createHandle(this._scope, await this._frame.extendInjectedScript(params.source, parseArgument(params.arg))) };
-  }
 }

src/protocol/channels.ts

@@ -1209,7 +1209,6 @@ export interface FrameChannel extends Channel {
   uncheck(params: FrameUncheckParams, metadata?: Metadata): Promise<FrameUncheckResult>;
   waitForFunction(params: FrameWaitForFunctionParams, metadata?: Metadata): Promise<FrameWaitForFunctionResult>;
   waitForSelector(params: FrameWaitForSelectorParams, metadata?: Metadata): Promise<FrameWaitForSelectorResult>;
-  extendInjectedScript(params: FrameExtendInjectedScriptParams, metadata?: Metadata): Promise<FrameExtendInjectedScriptResult>;
 }
 export type FrameLoadstateEvent = {
   add?: 'load' | 'domcontentloaded' | 'networkidle',
@@ -1669,16 +1668,6 @@ export type FrameWaitForSelectorOptions = {
 export type FrameWaitForSelectorResult = {
   element?: ElementHandleChannel,
 };
-export type FrameExtendInjectedScriptParams = {
-  source: string,
-  arg: SerializedArgument,
-};
-export type FrameExtendInjectedScriptOptions = {
-
-};
-export type FrameExtendInjectedScriptResult = {
-  handle: JSHandleChannel,
-};
 
 // ----------- Worker -----------
 export type WorkerInitializer = {

src/protocol/protocol.yml

@@ -1396,14 +1396,6 @@ Frame:
       returns:
         element: ElementHandle?
 
-    extendInjectedScript:
-      experimental: True
-      parameters:
-        source: string
-        arg: SerializedArgument
-      returns:
-        handle: JSHandle
-
   events:
 
     loadstate:

src/protocol/validator.ts

@@ -679,10 +679,6 @@ export function createScheme(tChannel: (name: string) => Validator): Scheme {
     timeout: tOptional(tNumber),
     state: tOptional(tEnum(['attached', 'detached', 'visible', 'hidden'])),
   });
-  scheme.FrameExtendInjectedScriptParams = tObject({
-    source: tString,
-    arg: tType('SerializedArgument'),
-  });
   scheme.WorkerEvaluateExpressionParams = tObject({
     expression: tString,
     isFunction: tBoolean,

src/utils/binaryPaths.ts

@@ -34,18 +34,11 @@ export function ffmpegExecutable(): string | undefined {
 }
 
 function pathToExecutable(relative: string[]): string | undefined {
-  const defaultPath = path.join(__dirname, '..', '..', ...relative);
-  const localPath = path.join(path.dirname(process.argv[0]), relative[relative.length - 1]);
   try {
+    const defaultPath = path.join(__dirname, '..', '..', ...relative);
     if (fs.existsSync(defaultPath))
       return defaultPath;
   } catch (e) {
   }
-
-  try {
-    if (fs.existsSync(localPath))
-      return localPath;
-  } catch (e) {
-  }
 }
 

test/trace.spec.ts

@@ -15,7 +15,7 @@
  */
 
 import { it, expect } from './fixtures';
-import type * as trace from '../types/trace';
+import type * as trace from '../src/trace/traceTypes';
 import * as path from 'path';
 import * as fs from 'fs';
 

types/trace.d.ts

@@ -36,7 +36,6 @@ let hadChanges = false;
   if (!fs.existsSync(typesDir))
     fs.mkdirSync(typesDir)
   writeFile(path.join(typesDir, 'protocol.d.ts'), fs.readFileSync(path.join(PROJECT_DIR, 'src', 'server', 'chromium', 'protocol.ts'), 'utf8'));
-  writeFile(path.join(typesDir, 'trace.d.ts'), fs.readFileSync(path.join(PROJECT_DIR, 'src', 'trace', 'traceTypes.ts'), 'utf8'));
   documentation = parseApi(path.join(PROJECT_DIR, 'docs', 'src', 'api'));
   documentation.filterForLanguage('js');
   documentation.copyDocsFromSuperclasses([]);