Removing an experimental feature (ampproject#13076)

gzgogo · Feb 14, 2018 · 7c30781 · 7c30781
1 parent 8b48793
commit 7c30781
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 71 deletions.
diff --git a/build-system/amp.extern.js b/build-system/amp.extern.js
@@ -425,3 +425,13 @@ var rtcResponseDef;
  * @type {!function(string):?}
  */
 AMP.require;
+
+/**
+ * TransitionDef function that accepts normtime, typically between 0 and 1 and
+ * performs an arbitrary animation action. Notice that sometimes normtime can
+ * dip above 1 or below 0. This is an acceptable case for some curves. The
+ * second argument is a boolean value that equals "true" for the completed
+ * transition and "false" for ongoing.
+ * @typedef {function(number, boolean):?|function(number):?}
+ */
+var TransitionDef;
diff --git a/build-system/amp.nti.extern.js b/build-system/amp.nti.extern.js
diff --git a/build-system/amp.oti.extern.js b/build-system/amp.oti.extern.js
diff --git a/build-system/tasks/compile.js b/build-system/tasks/compile.js
@@ -311,20 +311,6 @@ function compile(entryModuleFilenames, outputDir,
           'globalThis');
       compilerOptions.compilerFlags.conformance_configs =
           'build-system/conformance-config.textproto';
-
-      // TODO(aghassemi): Remove when NTI is the default.
-      if (argv.nti) {
-        compilerOptions.compilerFlags.new_type_inf = true;
-        compilerOptions.compilerFlags.jscomp_off.push(
-            'newCheckTypesExtraChecks');
-        compilerOptions.compilerFlags.externs.push(
-            'build-system/amp.nti.extern.js'
-        );
-      } else {
-        compilerOptions.compilerFlags.externs.push(
-            'build-system/amp.oti.extern.js'
-        );
-      }
     }
     if (argv.pseudo_names) {
       compilerOptions.compilerFlags.define.push('PSEUDO_NAMES=true');