Remove dead HostTargetController::installPerfIssuesBinding declaration

sammy-SC · meta-codesync[bot] · commit f6901706c341 · 2026-06-09T06:15:09.000-07:00
Summary:
`HostTargetController::installPerfIssuesBinding()` was declared in `jsinspector-modern/HostTarget.h` but had no definition anywhere and no callers. (`HostTargetController` is `final`, so the method is not an override.) A declared-but-never-defined non-virtual member cannot be invoked — any call would be a link error — so this is unreachable dead code. The unrelated, live `HostTarget::installPerfIssuesBinding` (a different class) is left intact.

Changelog: [Internal]

Differential Revision: D108012907
diff --git a/packages/react-native/ReactCommon/jsinspector-modern/HostTarget.h b/packages/react-native/ReactCommon/jsinspector-modern/HostTarget.h
@@ -252,13 +252,6 @@ class HostTargetController final {
 
   bool hasInstance() const;
 
-  /**
-   * [Experimental] Install a runtime binding subscribing to new Performance
-   * Issues, which we broadcast to the V2 Perf Monitor overlay via
-   * \ref HostTargetDelegate::unstable_onPerfIssueAdded.
-   */
-  void installPerfIssuesBinding();
-
   /**
    * Increments the target's pause overlay counter. The counter represents the
    * exact number of Agents that have (concurrently) requested the pause
diff --git a/scripts/cxx-api/api-snapshots/ReactAndroidDebugCxx.api b/scripts/cxx-api/api-snapshots/ReactAndroidDebugCxx.api
@@ -10511,7 +10511,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactAndroidNewarchCxx.api b/scripts/cxx-api/api-snapshots/ReactAndroidNewarchCxx.api
@@ -10162,7 +10162,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactAndroidReleaseCxx.api b/scripts/cxx-api/api-snapshots/ReactAndroidReleaseCxx.api
@@ -10364,7 +10364,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactAppleDebugCxx.api b/scripts/cxx-api/api-snapshots/ReactAppleDebugCxx.api
@@ -12405,7 +12405,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactAppleNewarchCxx.api b/scripts/cxx-api/api-snapshots/ReactAppleNewarchCxx.api
@@ -12118,7 +12118,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactAppleReleaseCxx.api b/scripts/cxx-api/api-snapshots/ReactAppleReleaseCxx.api
@@ -12268,7 +12268,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactCommonDebugCxx.api b/scripts/cxx-api/api-snapshots/ReactCommonDebugCxx.api
@@ -7566,7 +7566,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactCommonNewarchCxx.api b/scripts/cxx-api/api-snapshots/ReactCommonNewarchCxx.api
@@ -7419,7 +7419,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {
diff --git a/scripts/cxx-api/api-snapshots/ReactCommonReleaseCxx.api b/scripts/cxx-api/api-snapshots/ReactCommonReleaseCxx.api
@@ -7557,7 +7557,6 @@ class facebook::react::jsinspector_modern::HostTargetController {
   public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();
   public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();
   public void incrementPauseOverlayCounter();
-  public void installPerfIssuesBinding();
 }
 
 class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {

Original file line number	Diff line number	Diff line change
`@@ -10511,7 +10511,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`10511`	`10511`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`10512`	`10512`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`10513`	`10513`	`public void incrementPauseOverlayCounter();`
`10514`		`- public void installPerfIssuesBinding();`
`10515`	`10514`	`}`
`10516`	`10515`
`10517`	`10516`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -10162,7 +10162,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`10162`	`10162`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`10163`	`10163`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`10164`	`10164`	`public void incrementPauseOverlayCounter();`
`10165`		`- public void installPerfIssuesBinding();`
`10166`	`10165`	`}`
`10167`	`10166`
`10168`	`10167`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -10364,7 +10364,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`10364`	`10364`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`10365`	`10365`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`10366`	`10366`	`public void incrementPauseOverlayCounter();`
`10367`		`- public void installPerfIssuesBinding();`
`10368`	`10367`	`}`
`10369`	`10368`
`10370`	`10369`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -12405,7 +12405,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`12405`	`12405`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`12406`	`12406`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`12407`	`12407`	`public void incrementPauseOverlayCounter();`
`12408`		`- public void installPerfIssuesBinding();`
`12409`	`12408`	`}`
`12410`	`12409`
`12411`	`12410`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -12118,7 +12118,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`12118`	`12118`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`12119`	`12119`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`12120`	`12120`	`public void incrementPauseOverlayCounter();`
`12121`		`- public void installPerfIssuesBinding();`
`12122`	`12121`	`}`
`12123`	`12122`
`12124`	`12123`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -12268,7 +12268,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`12268`	`12268`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`12269`	`12269`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`12270`	`12270`	`public void incrementPauseOverlayCounter();`
`12271`		`- public void installPerfIssuesBinding();`
`12272`	`12271`	`}`
`12273`	`12272`
`12274`	`12273`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -7566,7 +7566,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`7566`	`7566`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`7567`	`7567`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`7568`	`7568`	`public void incrementPauseOverlayCounter();`
`7569`		`- public void installPerfIssuesBinding();`
`7570`	`7569`	`}`
`7571`	`7570`
`7572`	`7571`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -7419,7 +7419,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`7419`	`7419`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`7420`	`7420`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`7421`	`7421`	`public void incrementPauseOverlayCounter();`
`7422`		`- public void installPerfIssuesBinding();`
`7423`	`7422`	`}`
`7424`	`7423`
`7425`	`7424`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`
Original file line number	Diff line number	Diff line change
`@@ -7557,7 +7557,6 @@ class facebook::react::jsinspector_modern::HostTargetController {`
`7557`	`7557`	`public facebook::react::jsinspector_modern::HostTargetDelegate& getDelegate();`
`7558`	`7558`	`public facebook::react::jsinspector_modern::tracing::HostTracingProfile stopTracing();`
`7559`	`7559`	`public void incrementPauseOverlayCounter();`
`7560`		`- public void installPerfIssuesBinding();`
`7561`	`7560`	`}`
`7562`	`7561`
`7563`	`7562`	`class facebook::react::jsinspector_modern::HostTargetDelegate : public facebook::react::jsinspector_modern::LoadNetworkResourceDelegate {`