Title: [249274] trunk/Source/WebKit
Revision
249274
Author
[email protected]
Date
2019-08-29 08:27:27 -0700 (Thu, 29 Aug 2019)

Log Message

Use inline data member initialization in ProcessLauncher
https://bugs.webkit.org/show_bug.cgi?id=201247

Reviewed by Ryosuke Niwa.

* UIProcess/Launcher/ProcessLauncher.cpp:
(WebKit::ProcessLauncher::ProcessLauncher):
* UIProcess/Launcher/ProcessLauncher.h:

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (249273 => 249274)


--- trunk/Source/WebKit/ChangeLog	2019-08-29 15:25:32 UTC (rev 249273)
+++ trunk/Source/WebKit/ChangeLog	2019-08-29 15:27:27 UTC (rev 249274)
@@ -1,3 +1,14 @@
+2019-08-29  Chris Dumez  <[email protected]>
+
+        Use inline data member initialization in ProcessLauncher
+        https://bugs.webkit.org/show_bug.cgi?id=201247
+
+        Reviewed by Ryosuke Niwa.
+
+        * UIProcess/Launcher/ProcessLauncher.cpp:
+        (WebKit::ProcessLauncher::ProcessLauncher):
+        * UIProcess/Launcher/ProcessLauncher.h:
+
 2019-08-28  Tim Horton  <[email protected]>
 
         Reloading a web view with a fixed-width viewport and variable content width restores the previous page scale, shouldn't

Modified: trunk/Source/WebKit/UIProcess/Launcher/ProcessLauncher.cpp (249273 => 249274)


--- trunk/Source/WebKit/UIProcess/Launcher/ProcessLauncher.cpp	2019-08-29 15:25:32 UTC (rev 249273)
+++ trunk/Source/WebKit/UIProcess/Launcher/ProcessLauncher.cpp	2019-08-29 15:27:27 UTC (rev 249274)
@@ -35,9 +35,7 @@
 ProcessLauncher::ProcessLauncher(Client* client, LaunchOptions&& launchOptions)
     : m_client(client)
     , m_launchOptions(WTFMove(launchOptions))
-    , m_processIdentifier(0)
 {
-    m_isLaunching = true;
     tracePoint(ProcessLaunchStart);
     launchProcess();
 }

Modified: trunk/Source/WebKit/UIProcess/Launcher/ProcessLauncher.h (249273 => 249274)


--- trunk/Source/WebKit/UIProcess/Launcher/ProcessLauncher.h	2019-08-29 15:25:32 UTC (rev 249273)
+++ trunk/Source/WebKit/UIProcess/Launcher/ProcessLauncher.h	2019-08-29 15:27:27 UTC (rev 249274)
@@ -113,8 +113,8 @@
 #endif
 
     const LaunchOptions m_launchOptions;
-    bool m_isLaunching;
-    ProcessID m_processIdentifier;
+    bool m_isLaunching { true };
+    ProcessID m_processIdentifier { 0 };
 };
 
 } // namespace WebKit
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to