perf(common): make NgLocalization token tree-shakable

This commit updates the `NgLocalization` token to become tree-shakable (vs using a direct reference to that token in the `providers` section of the `CommonModule`). The `NgLocalization` token is used for apps that use i18n and for other apps it would be excluded from the bundle.
angular · Feb 17, 2022 · 9a6e233 · 9a6e233
1 parent db4a658
commit 9a6e233
Show file tree

Hide file tree

Showing 9 changed files with 19 additions and 110 deletions.
diff --git a/goldens/public-api/common/common.md b/goldens/public-api/common/common.md
@@ -507,6 +507,10 @@ export class NgLocaleLocalization extends NgLocalization {
 export abstract class NgLocalization {
     // (undocumented)
     abstract getPluralCategory(value: any, locale?: string): string;
+    // (undocumented)
+    static ɵfac: i0.ɵɵFactoryDeclaration<NgLocalization, never>;
+    // (undocumented)
+    static ɵprov: i0.ɵɵInjectableDeclaration<NgLocalization>;
 }
 
 // @public

diff --git a/goldens/size-tracking/aio-payloads.json b/goldens/size-tracking/aio-payloads.json
@@ -15,7 +15,7 @@
     "master": {
       "uncompressed": {
         "runtime": 4343,
-        "main": 450729,
+        "main": 449841,
         "polyfills": 37297,
         "styles": 70379,
         "light-theme": 77582,

diff --git a/goldens/size-tracking/integration-payloads.json b/goldens/size-tracking/integration-payloads.json
@@ -3,7 +3,7 @@
     "master": {
       "uncompressed": {
         "runtime": 1083,
-        "main": 127944,
+        "main": 126007,
         "polyfills": 37226
       }
     }
@@ -24,7 +24,7 @@
     "master": {
       "uncompressed": {
         "runtime": 1105,
-        "main": 133608,
+        "main": 132995,
         "polyfills": 37248
       }
     }
@@ -33,7 +33,7 @@
     "master": {
       "uncompressed": {
         "runtime": 929,
-        "main": 126275,
+        "main": 125690,
         "polyfills": 37933
       }
     }
@@ -42,7 +42,7 @@
     "master": {
       "uncompressed": {
         "runtime": 2835,
-        "main": 233348,
+        "main": 232795,
         "polyfills": 37244,
         "src_app_lazy_lazy_module_ts": 795
       }
@@ -52,7 +52,7 @@
     "master": {
       "uncompressed": {
         "runtime": 1063,
-        "main": 159637,
+        "main": 156872,
         "polyfills": 36975
       }
     }
@@ -61,7 +61,7 @@
     "master": {
       "uncompressed": {
         "runtime": 1070,
-        "main": 159755,
+        "main": 157264,
         "polyfills": 37242
       }
     }

diff --git a/packages/common/src/common_module.ts b/packages/common/src/common_module.ts
@@ -7,11 +7,12 @@
  */
 
 import {NgModule} from '@angular/core';
+
 import {COMMON_DIRECTIVES} from './directives/index';
-import {NgLocaleLocalization, NgLocalization} from './i18n/localization';
 import {COMMON_PIPES} from './pipes/index';
 
 
+
 // Note: This does not contain the location providers,
 // as they need some platform specific implementations to work.
 /**
@@ -30,9 +31,6 @@ import {COMMON_PIPES} from './pipes/index';
 @NgModule({
   declarations: [COMMON_DIRECTIVES, COMMON_PIPES],
   exports: [COMMON_DIRECTIVES, COMMON_PIPES],
-  providers: [
-    {provide: NgLocalization, useClass: NgLocaleLocalization},
-  ],
 })
 export class CommonModule {
 }
diff --git a/packages/common/src/i18n/localization.ts b/packages/common/src/i18n/localization.ts
@@ -6,19 +6,22 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {Inject, Injectable, LOCALE_ID} from '@angular/core';
+import {forwardRef, Inject, inject, Injectable, LOCALE_ID} from '@angular/core';
 
 import {getLocalePluralCase, Plural} from './locale_data_api';
 
-
 /**
  * @publicApi
  */
+@Injectable({
+  providedIn: 'root',
+  useClass: forwardRef(() => NgLocaleLocalization),
+  deps: [LOCALE_ID],
+})
 export abstract class NgLocalization {
   abstract getPluralCategory(value: any, locale?: string): string;
 }
 
-
 /**
  * Returns the plural category for a given value.
  * - "=value" when the case exists,

diff --git a/packages/core/test/bundling/animations/bundle.golden_symbols.json b/packages/core/test/bundling/animations/bundle.golden_symbols.json
@@ -254,18 +254,12 @@
   {
     "name": "LEAVE_TOKEN_REGEX"
   },
-  {
-    "name": "LOCALE_DATA"
-  },
   {
     "name": "LOCALE_ID2"
   },
   {
     "name": "LifecycleHooksFeature"
   },
-  {
-    "name": "LocaleDataIndex"
-  },
   {
     "name": "MODIFIER_KEYS"
   },
@@ -338,12 +332,6 @@
   {
     "name": "NULL_REMOVED_QUERIED_STATE"
   },
-  {
-    "name": "NgLocaleLocalization"
-  },
-  {
-    "name": "NgLocalization"
-  },
   {
     "name": "NgModuleFactory2"
   },
@@ -392,9 +380,6 @@
   {
     "name": "PlatformRef"
   },
-  {
-    "name": "Plural"
-  },
   {
     "name": "R3Injector"
   },
@@ -875,9 +860,6 @@
   {
     "name": "getLViewParent"
   },
-  {
-    "name": "getLocaleData"
-  },
   {
     "name": "getNativeByTNode"
   },
@@ -1103,9 +1085,6 @@
   {
     "name": "listenOnPlayer"
   },
-  {
-    "name": "locale_en_default"
-  },
   {
     "name": "lookupTokenUsingModuleInjector"
   },
@@ -1373,9 +1352,6 @@
   {
     "name": "transition"
   },
-  {
-    "name": "u"
-  },
   {
     "name": "unwrapRNode"
   },

diff --git a/packages/core/test/bundling/forms_reactive/bundle.golden_symbols.json b/packages/core/test/bundling/forms_reactive/bundle.golden_symbols.json
@@ -236,18 +236,12 @@
   {
     "name": "KeyEventsPlugin"
   },
-  {
-    "name": "LOCALE_DATA"
-  },
   {
     "name": "LOCALE_ID2"
   },
   {
     "name": "LifecycleHooksFeature"
   },
-  {
-    "name": "LocaleDataIndex"
-  },
   {
     "name": "MODIFIER_KEYS"
   },
@@ -341,12 +335,6 @@
   {
     "name": "NgForOf"
   },
-  {
-    "name": "NgLocaleLocalization"
-  },
-  {
-    "name": "NgLocalization"
-  },
   {
     "name": "NgModuleFactory2"
   },
@@ -389,9 +377,6 @@
   {
     "name": "PlatformRef"
   },
-  {
-    "name": "Plural"
-  },
   {
     "name": "R3Injector"
   },
@@ -920,9 +905,6 @@
   {
     "name": "getLViewParent"
   },
-  {
-    "name": "getLocaleData"
-  },
   {
     "name": "getNativeByTNode"
   },
@@ -1220,9 +1202,6 @@
   {
     "name": "leaveViewLight"
   },
-  {
-    "name": "locale_en_default"
-  },
   {
     "name": "lookupTokenUsingModuleInjector"
   },
@@ -1532,9 +1511,6 @@
   {
     "name": "trackByIdentity"
   },
-  {
-    "name": "u"
-  },
   {
     "name": "unwrapRNode"
   },

diff --git a/packages/core/test/bundling/forms_template_driven/bundle.golden_symbols.json b/packages/core/test/bundling/forms_template_driven/bundle.golden_symbols.json
@@ -227,18 +227,12 @@
   {
     "name": "KeyEventsPlugin"
   },
-  {
-    "name": "LOCALE_DATA"
-  },
   {
     "name": "LOCALE_ID2"
   },
   {
     "name": "LifecycleHooksFeature"
   },
-  {
-    "name": "LocaleDataIndex"
-  },
   {
     "name": "MODIFIER_KEYS"
   },
@@ -332,12 +326,6 @@
   {
     "name": "NgForm"
   },
-  {
-    "name": "NgLocaleLocalization"
-  },
-  {
-    "name": "NgLocalization"
-  },
   {
     "name": "NgModel"
   },
@@ -386,9 +374,6 @@
   {
     "name": "PlatformRef"
   },
-  {
-    "name": "Plural"
-  },
   {
     "name": "R3Injector"
   },
@@ -893,9 +878,6 @@
   {
     "name": "getLViewParent"
   },
-  {
-    "name": "getLocaleData"
-  },
   {
     "name": "getNativeByTNode"
   },
@@ -1187,9 +1169,6 @@
   {
     "name": "leaveViewLight"
   },
-  {
-    "name": "locale_en_default"
-  },
   {
     "name": "lookupTokenUsingModuleInjector"
   },
@@ -1517,9 +1496,6 @@
   {
     "name": "trackByIdentity"
   },
-  {
-    "name": "u"
-  },
   {
     "name": "unwrapRNode"
   },