perf(common): make NgLocalization token tree-shakable (angular#45118)

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. PR Close angular#45118
AndrewKushnir · Mar 1, 2022 · f752f2f · f752f2f
1 parent e8fd452
commit f752f2f
Show file tree

Hide file tree

Showing 9 changed files with 18 additions and 109 deletions.
diff --git a/goldens/public-api/common/common.md b/goldens/public-api/common/common.md
@@ -503,6 +503,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": 450840,
+        "main": 450239,
         "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": 126218,
+        "main": 124282,
         "polyfills": 37226
       }
     }
@@ -24,7 +24,7 @@
     "master": {
       "uncompressed": {
         "runtime": 1105,
-        "main": 131882,
+        "main": 131270,
         "polyfills": 37248
       }
     }
@@ -33,7 +33,7 @@
     "master": {
       "uncompressed": {
         "runtime": 929,
-        "main": 124544,
+        "main": 123958,
         "polyfills": 37933
       }
     }
@@ -42,7 +42,7 @@
     "master": {
       "uncompressed": {
         "runtime": 2835,
-        "main": 231381,
+        "main": 230780,
         "polyfills": 37244,
         "src_app_lazy_lazy_module_ts": 795
       }
@@ -52,7 +52,7 @@
     "master": {
       "uncompressed": {
         "runtime": 1063,
-        "main": 158556,
+        "main": 155607,
         "polyfills": 36975
       }
     }
@@ -61,7 +61,7 @@
     "master": {
       "uncompressed": {
         "runtime": 1070,
-        "main": 158300,
+        "main": 155563,
         "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
@@ -10,15 +10,18 @@ import {Inject, Injectable, LOCALE_ID} from '@angular/core';
 
 import {getLocalePluralCase, Plural} from './locale_data_api';
 
-
 /**
  * @publicApi
  */
+@Injectable({
+  providedIn: 'root',
+  useFactory: (locale: string) => new NgLocaleLocalization(locale),
+  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
@@ -242,18 +242,12 @@
   {
     "name": "LEAVE_TOKEN_REGEX"
   },
-  {
-    "name": "LOCALE_DATA"
-  },
   {
     "name": "LOCALE_ID2"
   },
   {
     "name": "LifecycleHooksFeature"
   },
-  {
-    "name": "LocaleDataIndex"
-  },
   {
     "name": "MODIFIER_KEYS"
   },
@@ -326,12 +320,6 @@
   {
     "name": "NULL_REMOVED_QUERIED_STATE"
   },
-  {
-    "name": "NgLocaleLocalization"
-  },
-  {
-    "name": "NgLocalization"
-  },
   {
     "name": "NgModuleRef"
   },
@@ -374,9 +362,6 @@
   {
     "name": "PlatformRef"
   },
-  {
-    "name": "Plural"
-  },
   {
     "name": "R3Injector"
   },
@@ -848,9 +833,6 @@
   {
     "name": "getLViewParent"
   },
-  {
-    "name": "getLocaleData"
-  },
   {
     "name": "getNativeByTNode"
   },
@@ -1085,9 +1067,6 @@
   {
     "name": "listenOnPlayer"
   },
-  {
-    "name": "locale_en_default"
-  },
   {
     "name": "lookupTokenUsingModuleInjector"
   },
@@ -1352,9 +1331,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
@@ -221,18 +221,12 @@
   {
     "name": "KeyEventsPlugin"
   },
-  {
-    "name": "LOCALE_DATA"
-  },
   {
     "name": "LOCALE_ID2"
   },
   {
     "name": "LifecycleHooksFeature"
   },
-  {
-    "name": "LocaleDataIndex"
-  },
   {
     "name": "MODIFIER_KEYS"
   },
@@ -326,12 +320,6 @@
   {
     "name": "NgForOf"
   },
-  {
-    "name": "NgLocaleLocalization"
-  },
-  {
-    "name": "NgLocalization"
-  },
   {
     "name": "NgModuleFactory2"
   },
@@ -374,9 +362,6 @@
   {
     "name": "PlatformRef"
   },
-  {
-    "name": "Plural"
-  },
   {
     "name": "R3Injector"
   },
@@ -902,9 +887,6 @@
   {
     "name": "getLViewParent"
   },
-  {
-    "name": "getLocaleData"
-  },
   {
     "name": "getNativeByTNode"
   },
@@ -1205,9 +1187,6 @@
   {
     "name": "leaveViewLight"
   },
-  {
-    "name": "locale_en_default"
-  },
   {
     "name": "lookupTokenUsingModuleInjector"
   },
@@ -1517,9 +1496,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
@@ -212,18 +212,12 @@
   {
     "name": "KeyEventsPlugin"
   },
-  {
-    "name": "LOCALE_DATA"
-  },
   {
     "name": "LOCALE_ID2"
   },
   {
     "name": "LifecycleHooksFeature"
   },
-  {
-    "name": "LocaleDataIndex"
-  },
   {
     "name": "MODIFIER_KEYS"
   },
@@ -317,12 +311,6 @@
   {
     "name": "NgForm"
   },
-  {
-    "name": "NgLocaleLocalization"
-  },
-  {
-    "name": "NgLocalization"
-  },
   {
     "name": "NgModel"
   },
@@ -371,9 +359,6 @@
   {
     "name": "PlatformRef"
   },
-  {
-    "name": "Plural"
-  },
   {
     "name": "R3Injector"
   },
@@ -872,9 +857,6 @@
   {
     "name": "getLViewParent"
   },
-  {
-    "name": "getLocaleData"
-  },
   {
     "name": "getNativeByTNode"
   },
@@ -1169,9 +1151,6 @@
   {
     "name": "leaveViewLight"
   },
-  {
-    "name": "locale_en_default"
-  },
   {
     "name": "lookupTokenUsingModuleInjector"
   },
@@ -1499,9 +1478,6 @@
   {
     "name": "trackByIdentity"
   },
-  {
-    "name": "u"
-  },
   {
     "name": "unwrapRNode"
   },