Remove network experiment and experimental icon

Author: huntie

front_end/core/rn_experiments/experimentsImpl.ts

@@ -184,10 +184,3 @@ Instance.register({
   unstable: false,
   enabledByDefault: ({ isReactNativeEntryPoint }) => isReactNativeEntryPoint,
 });
-
-Instance.register({
-  name: RNExperimentName.ENABLE_NETWORK_PANEL,
-  title: 'Enable Network panel',
-  unstable: true,
-  enabledByDefault: () => false,
-});

front_end/core/root/Runtime.ts

@@ -305,7 +305,6 @@ export const experiments = new ExperimentsSupport();
 export enum RNExperimentName {
   REACT_NATIVE_SPECIFIC_UI = 'react-native-specific-ui',
   JS_HEAP_PROFILER_ENABLE = 'js-heap-profiler-enable',
-  ENABLE_NETWORK_PANEL = 'enable-network-panel',
 }
 
 export enum ConditionName {
@@ -340,7 +339,6 @@ export const enum ExperimentName {
   JS_HEAP_PROFILER_ENABLE = RNExperimentName.JS_HEAP_PROFILER_ENABLE,
   REACT_NATIVE_SPECIFIC_UI = RNExperimentName.REACT_NATIVE_SPECIFIC_UI,
   NOT_REACT_NATIVE_SPECIFIC_UI = '!' + RNExperimentName.REACT_NATIVE_SPECIFIC_UI,
-  ENABLE_NETWORK_PANEL = RNExperimentName.ENABLE_NETWORK_PANEL,
 }
 
 export enum GenAiEnterprisePolicyValue {
@@ -515,7 +513,6 @@ export type Condition = (config?: HostConfig) => boolean;
 export const conditions = {
   canDock: () => Boolean(Runtime.queryParam('can_dock')),
   notSourcesHideAddFolder: () => Boolean(Runtime.queryParam(ConditionName.NOT_SOURCES_HIDE_ADD_FOLDER)),
-  reactNativeUnstableNetworkPanel: () =>
-      Boolean(Runtime.queryParam(ConditionName.REACT_NATIVE_UNSTABLE_NETWORK_PANEL)) ||
-      experiments.isEnabled(ExperimentName.ENABLE_NETWORK_PANEL),
+  reactNativeUnstableNetworkPanel: () => true,
+  reactNativeExpoNetworkPanel: () => Boolean(Runtime.queryParam(ConditionName.REACT_NATIVE_UNSTABLE_NETWORK_PANEL)),
 };

front_end/entrypoints/rn_fusebox/FuseboxFeatureObserver.ts

@@ -5,7 +5,6 @@
 
 import type * as Common from '../../core/common/common.js';
 import * as i18n from '../../core/i18n/i18n.js';
-import * as Root from '../../core/root/root.js';
 import * as SDK from '../../core/sdk/sdk.js';
 import type * as Protocol from '../../generated/protocol.js';
 import * as UI from '../../ui/legacy/legacy.js';
@@ -17,10 +16,6 @@ import {FuseboxWindowTitleManager} from './FuseboxWindowTitleManager.js';
 const {html, render} = Lit;
 
 const UIStrings = {
-  /**
-   * @description Message for the "settings changed" banner shown when a reload is required for the Network panel.
-   */
-  reloadRequiredForNetworkPanelMessage: 'The Network panel is now available for dogfooding. Please reload to access it.',
   /**
    * @description Title shown when Network inspection is disabled due to multiple React Native hosts.
    */
@@ -79,16 +74,12 @@ export class FuseboxFeatureObserver implements
   #handleMetadataUpdated(
       event: Common.EventTarget.EventTargetEvent<Protocol.ReactNativeApplication.MetadataUpdatedEvent>): void {
     // eslint-disable-next-line @typescript-eslint/naming-convention
-    const {unstable_isProfilingBuild, unstable_networkInspectionEnabled} = event.data;
+    const {unstable_isProfilingBuild} = event.data;
 
     if (unstable_isProfilingBuild) {
       FuseboxWindowTitleManager.instance().setSuffix('[PROFILING]');
       this.#hideUnsupportedFeaturesForProfilingBuilds();
     }
-
-    if (unstable_networkInspectionEnabled) {
-      this.#ensureNetworkPanelEnabled();
-    }
   }
 
   #handleSystemStateChanged(
@@ -127,21 +118,6 @@ export class FuseboxFeatureObserver implements
       });
   }
 
-  #ensureNetworkPanelEnabled(): void {
-    if (Root.Runtime.experiments.isEnabled(Root.Runtime.ExperimentName.ENABLE_NETWORK_PANEL)) {
-      return;
-    }
-
-    Root.Runtime.experiments.setEnabled(
-        Root.Runtime.ExperimentName.ENABLE_NETWORK_PANEL,
-        true,
-    );
-
-    UI.InspectorView?.InspectorView?.instance()?.displayReloadRequiredWarning(
-        i18nString(UIStrings.reloadRequiredForNetworkPanelMessage),
-    );
-  }
-
   #disableSingleHostOnlyFeatures(): void {
     if (this.#singleHostFeaturesDisabled) {
       return;

front_end/panels/network/NetworkItemView.ts

@@ -151,18 +151,12 @@ export class NetworkItemView extends UI.TabbedPane.TabbedPane {
   private readonly responseView: RequestResponseView|undefined;
   private cookiesView: RequestCookiesView|null;
   private initialTab?: NetworkForward.UIRequestLocation.UIRequestTabs;
-  private readonly isReactNative: boolean = false;
 
   constructor(
       request: SDK.NetworkRequest.NetworkRequest, calculator: NetworkTimeCalculator,
       initialTab?: NetworkForward.UIRequestLocation.UIRequestTabs) {
     super();
 
-    // [RN] Used to scope down available features for React Native targets
-    this.isReactNative = Root.Runtime.experiments.isEnabled(
-      Root.Runtime.ExperimentName.REACT_NATIVE_SPECIFIC_UI,
-    );
-
     this.requestInternal = request;
     this.element.classList.add('network-item-view');
     this.headerElement().setAttribute('jslog', `${VisualLogging.toolbar('request-details').track({
@@ -230,7 +224,7 @@ export class NetworkItemView extends UI.TabbedPane.TabbedPane {
       }
     }
 
-    if (!this.isReactNative || Root.Runtime.experiments.isEnabled(Root.Runtime.RNExperimentName.ENABLE_NETWORK_PANEL)) {
+    if (!Root.Runtime.conditions.reactNativeExpoNetworkPanel()) {
       this.appendTab(
           NetworkForward.UIRequestLocation.UIRequestTabs.INITIATOR, i18nString(UIStrings.initiator),
           new RequestInitiatorView(request), i18nString(UIStrings.requestInitiatorCallStack));

front_end/panels/network/NetworkPanel.test.ts

@@ -70,7 +70,6 @@ describeWithMockConnection('NetworkPanel', () => {
     registerNoopActions(['inspector-main.reload']);
     UI.ActionRegistration.maybeRemoveActionExtension('network.toggle-recording');
     UI.ActionRegistration.maybeRemoveActionExtension('network.clear');
-    Root.Runtime.experiments.register(Root.Runtime.RNExperimentName.ENABLE_NETWORK_PANEL, 'Network for test');
     await import('./network-meta.js');
     createTarget();
     const dummyStorage = new Common.Settings.SettingsStorage({});

front_end/panels/network/network-meta.ts

@@ -24,9 +24,9 @@ const UIStrings = {
    */
   network: 'Network',
   /**
-   *@description Title of the Network tool (Expo, unstable)
+   *@description Title of the Network tool (Expo implementation)
    */
-  networkExpoUnstable: 'Network (Expo, unstable)',
+  networkExpoUnstable: 'Expo Network',
   /**
    *@description Command for showing the 'Network request blocking' tool
    */
@@ -167,11 +167,10 @@ UI.ViewManager.registerViewExtension({
   location: UI.ViewManager.ViewLocationValues.PANEL,
   id: 'network',
   commandPrompt: i18nLazyString(UIStrings.showNetwork),
-  title: () => Root.Runtime.experiments.isEnabled(Root.Runtime.RNExperimentName.ENABLE_NETWORK_PANEL) ?
-      i18nString(UIStrings.network) :
-      i18nString(UIStrings.networkExpoUnstable),
+  title: () => Root.Runtime.conditions.reactNativeExpoNetworkPanel() ?
+      i18nString(UIStrings.networkExpoUnstable) :
+      i18nString(UIStrings.network),
   order: 40,
-  isPreviewFeature: true,
   condition: Root.Runtime.conditions.reactNativeUnstableNetworkPanel,
   async loadView() {
     const Network = await loadNetworkModule();