clean comments

Author: cnishina

Diff for: lib/browser.ts

@@ -489,7 +489,6 @@ export class ProtractorBrowser extends AbstractExtendedWebDriver {
       script = 'return (' + script + ').apply(null, arguments);';
     }
 
-    // TODO(selenium4): fix promise cast.
     // TODO(selenium4): schedule does not exist on driver. Should use execute instead.
     return (this.driver as any)
         .execute(new Command(CommandName.EXECUTE_SCRIPT)

Diff for: lib/driverProviders/attachSession.ts

@@ -40,7 +40,6 @@ export class AttachSession extends DriverProvider {
     const executor = new http.Executor(httpClient);
     const session = new Session(this.config_.seleniumSessionId, null);
 
-    // const newDriver = WebDriver.attachToSession(executor, this.config_.seleniumSessionId, null);
     const newDriver = new WebDriver(session, executor);
     this.drivers_.push(newDriver);
     return newDriver;

Diff for: lib/launcher.ts

@@ -171,7 +171,6 @@ let initFn = async function(configFile: string, additionalConfig: Config) {
   });
 
   process.on('unhandledRejection', (reason: Error | any, p: Promise<any>) => {
-    // let e = new Error();
     if (reason.stack.match('angular testability are undefined') ||
         reason.stack.match('angular is not defined')) {
       logger.warn(

Diff for: lib/runner.ts

@@ -256,7 +256,8 @@ export class Runner extends EventEmitter {
     }
 
     await browser_.waitForAngularEnabled(initProperties.waitForAngularEnabled);
-    await driver.manage().timeouts().setScriptTimeout(initProperties.allScriptsTimeout || 0);
+    // TODO(selenium4): Options does not have a setScriptTimeout method.
+    await(driver.manage() as any).setScriptTimeout(initProperties.allScriptsTimeout || 0);
 
     browser_.getProcessedConfig = () => {
       return Promise.resolve(config);

Diff for: scripts/test.js

@@ -50,10 +50,7 @@ const passingTests = [
   // Dependency tests
   'node node_modules/jasmine/bin/jasmine.js JASMINE_CONFIG_PATH=scripts/dependency_test.json',
   // Typings tests
-<<<<<<< HEAD
-=======
-  // TODO(selenium4): not sure why this does not work.
->>>>>>> deps(selenium): upgrade to selenium 4
+  // TODO(selenium4): consider rewriting this test.
   // 'node spec/install/test.js'
 ];