From 499d05ddeedc3db797509e44405df5e77b10df13 Mon Sep 17 00:00:00 2001
From: Olivier Combe <olivier.combe@gmail.com>
Date: Fri, 18 Aug 2017 15:31:32 +0200
Subject: [PATCH] refactor(compiler): remove option `useDebug` (#18778)

BREAKING CHANGE: the option `useDebug` for the compiler has been removed as it had no effect and was deprecated since v4.

PR Close #18778
---
 packages/core/src/linker/compiler.ts                      | 4 ----
 packages/core/testing/src/test_bed.ts                     | 3 +--
 packages/platform-browser-dynamic/src/compiler_factory.ts | 1 -
 packages/tsc-wrapped/src/options.ts                       | 3 ---
 tools/public_api_guard/core/core.d.ts                     | 1 -
 5 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/packages/core/src/linker/compiler.ts b/packages/core/src/linker/compiler.ts
index 5770e1f9f7..5149882e5d 100644
--- a/packages/core/src/linker/compiler.ts
+++ b/packages/core/src/linker/compiler.ts
@@ -96,10 +96,6 @@ export class Compiler {
  * @experimental
  */
 export type CompilerOptions = {
-  /**
-   * @deprecated since v4 this option has no effect anymore.
-   */
-  useDebug?: boolean,
   useJit?: boolean,
   defaultEncapsulation?: ViewEncapsulation,
   providers?: StaticProvider[],
diff --git a/packages/core/testing/src/test_bed.ts b/packages/core/testing/src/test_bed.ts
index f6bfb0ddbb..7368d95b03 100644
--- a/packages/core/testing/src/test_bed.ts
+++ b/packages/core/testing/src/test_bed.ts
@@ -330,8 +330,7 @@ export class TestBed implements Injector {
 
     const compilerFactory: TestingCompilerFactory =
         this.platform.injector.get(TestingCompilerFactory);
-    this._compiler =
-        compilerFactory.createTestingCompiler(this._compilerOptions.concat([{useDebug: true}]));
+    this._compiler = compilerFactory.createTestingCompiler(this._compilerOptions);
     this._compiler.loadAotSummaries(this._aotSummaries);
     this._moduleOverrides.forEach((entry) => this._compiler.overrideModule(entry[0], entry[1]));
     this._componentOverrides.forEach(
diff --git a/packages/platform-browser-dynamic/src/compiler_factory.ts b/packages/platform-browser-dynamic/src/compiler_factory.ts
index 1ffb7a839f..9581f26262 100644
--- a/packages/platform-browser-dynamic/src/compiler_factory.ts
+++ b/packages/platform-browser-dynamic/src/compiler_factory.ts
@@ -157,7 +157,6 @@ export class JitCompilerFactory implements CompilerFactory {
   private _defaultOptions: CompilerOptions[];
   constructor(defaultOptions: CompilerOptions[]) {
     const compilerOptions: CompilerOptions = {
-      useDebug: isDevMode(),
       useJit: true,
       defaultEncapsulation: ViewEncapsulation.Emulated,
       missingTranslation: MissingTranslationStrategy.Warning,
diff --git a/packages/tsc-wrapped/src/options.ts b/packages/tsc-wrapped/src/options.ts
index 8b1020105d..f58ee7d308 100644
--- a/packages/tsc-wrapped/src/options.ts
+++ b/packages/tsc-wrapped/src/options.ts
@@ -76,9 +76,6 @@ interface Options extends ts.CompilerOptions {
   // Print extra information while running the compiler
   trace?: boolean;
 
-  /** @deprecated since v4 this option has no effect anymore. */
-  debug?: boolean;
-
   // Whether to enable support for <template> and the template attribute (true by default)
   enableLegacyTemplate?: boolean;
 
diff --git a/tools/public_api_guard/core/core.d.ts b/tools/public_api_guard/core/core.d.ts
index 54b90a4e97..bfb1b2ba7d 100644
--- a/tools/public_api_guard/core/core.d.ts
+++ b/tools/public_api_guard/core/core.d.ts
@@ -195,7 +195,6 @@ export declare abstract class CompilerFactory {
 
 /** @experimental */
 export declare type CompilerOptions = {
-    /** @deprecated */ useDebug?: boolean;
     useJit?: boolean;
     defaultEncapsulation?: ViewEncapsulation;
     providers?: StaticProvider[];