[VE] Fail E2E test on Unknown VE context

Report the error when running E2E test, that will catch some of the
issue before getting merged.

Bug: none
Change-Id: I9cc984edf9ec61c60b175b5d27bc444151a9ae69
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6352082
Reviewed-by: Liviu Rau <[email protected]>
Reviewed-by: Benedikt Meurer <[email protected]>
Auto-Submit: Nikolay Vitkov <[email protected]>
Commit-Queue: Nikolay Vitkov <[email protected]>

Author: Lightning00Blade

front_end/core/common/Settings.ts

@@ -194,7 +194,7 @@ export class Settings {
    */
   createSetting<T>(key: string, defaultValue: T, storageType?: SettingStorageType): Setting<T> {
     const storage = this.storageFromType(storageType);
-    let setting = (this.#registry.get(key) as Setting<T>);
+    let setting = this.#registry.get(key) as Setting<T>;
     if (!setting) {
       setting = new Setting(key, defaultValue, this.#eventSupport, storage, this.#logSettingAccess);
       this.#registry.set(key, setting);

front_end/ui/visual_logging/KnownContextValues.ts

@@ -914,6 +914,7 @@ export const knownContextValues = new Set([
   'copy-link-address',
   'copy-object',
   'copy-outer-html',
+  'copy-payload',
   'copy-primitive',
   'copy-property',
   'copy-property-path',
@@ -1196,6 +1197,7 @@ export const knownContextValues = new Set([
   'disable-paused-state-overlay',
   'disable-self-xss-warning',
   'disabled',
+  'disallowed-select-descendants-details',
   'disconnect-from-network',
   'display',
   'display-override',
@@ -2031,6 +2033,7 @@ export const knownContextValues = new Set([
   'lighthouse.audit.bypass',
   'lighthouse.audit.canonical',
   'lighthouse.audit.charset',
+  'lighthouse.audit.clickjacking-mitigation',
   'lighthouse.audit.color-contrast',
   'lighthouse.audit.crawlable-anchors',
   'lighthouse.audit.critical-request-chains',
@@ -2060,6 +2063,7 @@ export const knownContextValues = new Set([
   'lighthouse.audit.form-field-multiple-labels',
   'lighthouse.audit.frame-title',
   'lighthouse.audit.geolocation-on-start',
+  'lighthouse.audit.has-hsts',
   'lighthouse.audit.heading-order',
   'lighthouse.audit.hreflang',
   'lighthouse.audit.html-has-lang',
@@ -2113,6 +2117,7 @@ export const knownContextValues = new Set([
   'lighthouse.audit.object-alt',
   'lighthouse.audit.offscreen-content-hidden',
   'lighthouse.audit.offscreen-images',
+  'lighthouse.audit.origin-isolation',
   'lighthouse.audit.paste-preventing-inputs',
   'lighthouse.audit.prioritize-lcp-image',
   'lighthouse.audit.redirects',
@@ -2197,6 +2202,7 @@ export const knownContextValues = new Set([
   'linear-memory-inspector.toggle-value-settings',
   'linear-memory-inspector.value-type-mode',
   'linear-memory-inspector.viewer',
+  'link',
   'link-in-explanation',
   'linux',
   'list-style',
@@ -2523,6 +2529,7 @@ export const knownContextValues = new Set([
   'only-show-cookies-with-issues',
   'only-show-third-party',
   'opacity',
+  'open',
   'open-ai-settings',
   'open-elements-panel',
   'open-folder',
@@ -2611,6 +2618,7 @@ export const knownContextValues = new Set([
   'parentTreeItem',
   'parse',
   'partition-key-site',
+  'partitioning-blob-url-details',
   'passive',
   'paste',
   'path',

front_end/ui/visual_logging/LoggingConfig.ts

@@ -2,8 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+import * as Host from '../../core/host/host.js';
 import * as Root from '../../core/root/root.js';
 
+import {DebugLoggingFormat} from './Debugging.js';
 import {knownContextValues} from './KnownContextValues.js';
 
 const LOGGING_ATTRIBUTE = 'jslog';
@@ -126,7 +128,8 @@ function checkContextValue(context: string|number|undefined): void {
       reportedUnknownVeContext.has(context)) {
     return;
   }
-  if (Root.Runtime.Runtime.queryParam('debugFrontend')) {
+  if (Root.Runtime.Runtime.queryParam('debugFrontend') || Host.InspectorFrontendHost.isUnderTest() ||
+      localStorage.getItem('veDebugLoggingEnabled') === DebugLoggingFormat.TEST) {
     const stack = (new Error().stack || '').split('\n').slice(3).join('\n');
     console.error(`Unknown VE context: ${context}${stack}`);
   }

test/e2e_non_hosted/shared/frontend-helper.ts

@@ -101,7 +101,9 @@ export interface DevtoolsSettings {
 
 export const DEFAULT_DEVTOOLS_SETTINGS = {
   enabledDevToolsExperiments: [],
-  devToolsSettings: {},
+  devToolsSettings: {
+    isUnderTest: true,
+  },
   dockingMode: 'UNDOCKED',
 };