refactor(core): move outdated comment to view.ts (angular#44099)

remove the comment suggesting to use a const enum for ViewEncapsulation in
the renderer3/definitions.ts file and add a similar comment in the
view.ts file in which the enum is defined

Note: the new comment does not contain the suggestion of changing `None`
to `0` as it is unclear what benefits that would bring (for more info
see: angular#44099 (comment))

PR Close angular#44099

Author: dario-piotrowicz

packages/core/src/metadata/view.ts

@@ -19,6 +19,9 @@
  * @publicApi
  */
 export enum ViewEncapsulation {
+  // TODO: consider making `ViewEncapsulation` a `const enum` instead. See
+  // https://github.com/angular/angular/issues/44119 for additional information.
+
   /**
    * Emulate `Native` scoping of styles by adding an attribute containing surrogate id to the Host
    * Element and pre-processing the style rules provided via {@link Component#styles styles} or

packages/core/src/render3/definition.ts

@@ -315,8 +315,6 @@ export function ɵɵdefineComponent<T>(componentDefinition: {
       viewQuery: componentDefinition.viewQuery || null,
       features: componentDefinition.features as DirectiveDefFeature[] || null,
       data: componentDefinition.data || {},
-      // TODO(misko): convert ViewEncapsulation into const enum so that it can be used
-      // directly in the next line. Also `None` should be 0 not 2.
       encapsulation: componentDefinition.encapsulation || ViewEncapsulation.Emulated,
       id: 'c',
       styles: componentDefinition.styles || EMPTY_ARRAY,