Bug 1702756 - Move RemoteLookAndFeel::sCachedLookAndFeelData out of the header. r=mstange

No reason it needs to be there. This was just drive-by.

Depends on D110670

Differential Revision: https://phabricator.services.mozilla.com/D110671

widget/RemoteLookAndFeel.cpp

@@ -23,6 +23,14 @@
 
 namespace mozilla::widget {
 
+// A cached copy of the data extracted by ExtractData.
+//
+// Storing this lets us avoid doing most of the work of ExtractData each
+// time we create a new content process.
+//
+// Only used in the parent process.
+static StaticAutoPtr<FullLookAndFeel> sCachedLookAndFeelData;
+
 RemoteLookAndFeel::RemoteLookAndFeel(FullLookAndFeel&& aData)
     : mTables(std::move(aData.tables())) {
   MOZ_ASSERT(XRE_IsContentProcess(),
@@ -242,6 +250,4 @@ void RemoteLookAndFeel::ClearCachedData() {
   sCachedLookAndFeelData = nullptr;
 }
 
-StaticAutoPtr<FullLookAndFeel> RemoteLookAndFeel::sCachedLookAndFeelData;
-
 }  // namespace mozilla::widget

widget/RemoteLookAndFeel.h

@@ -56,14 +56,6 @@ class RemoteLookAndFeel final : public nsXPLookAndFeel {
 
  private:
   LookAndFeelTables mTables;
-
-  // A cached copy of the data extracted by ExtractData.
-  //
-  // Storing this lets us avoid doing most of the work of ExtractData each
-  // time we create a new content process.
-  //
-  // Only used in the parent process.
-  static StaticAutoPtr<FullLookAndFeel> sCachedLookAndFeelData;
 };
 
 }  // namespace mozilla::widget