Title: [240747] trunk/Source/WebKit
Revision
240747
Author
[email protected]
Date
2019-01-30 17:55:27 -0800 (Wed, 30 Jan 2019)

Log Message

Add default constructor for NetworkActivityTracker
https://bugs.webkit.org/show_bug.cgi?id=194058
<rdar://problem/47685457>

Reviewed by Chris Dumez.

ResourceNetworkActivityTracker has a default constructor. It also has
a NetworkActivityTracker data member, which does not have a default
constructor. This will cause some compilers to complain -- they can't
default-create a ResourceNetworkActivityTracker because they can't
default-create its data members. Address this by adding a default
constructor for NetworkActivityTracker.

* NetworkProcess/NetworkActivityTracker.h:

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (240746 => 240747)


--- trunk/Source/WebKit/ChangeLog	2019-01-31 01:52:55 UTC (rev 240746)
+++ trunk/Source/WebKit/ChangeLog	2019-01-31 01:55:27 UTC (rev 240747)
@@ -1,3 +1,20 @@
+2019-01-30  Keith Rollin  <[email protected]>
+
+        Add default constructor for NetworkActivityTracker
+        https://bugs.webkit.org/show_bug.cgi?id=194058
+        <rdar://problem/47685457>
+
+        Reviewed by Chris Dumez.
+
+        ResourceNetworkActivityTracker has a default constructor. It also has
+        a NetworkActivityTracker data member, which does not have a default
+        constructor. This will cause some compilers to complain -- they can't
+        default-create a ResourceNetworkActivityTracker because they can't
+        default-create its data members. Address this by adding a default
+        constructor for NetworkActivityTracker.
+
+        * NetworkProcess/NetworkActivityTracker.h:
+
 2019-01-30  Daniel Steffen  <[email protected]>
 
         <rdar://problem/29471922> Safari should switch from the legacy denap SPI to handling vouchers

Modified: trunk/Source/WebKit/NetworkProcess/NetworkActivityTracker.h (240746 => 240747)


--- trunk/Source/WebKit/NetworkProcess/NetworkActivityTracker.h	2019-01-31 01:52:55 UTC (rev 240746)
+++ trunk/Source/WebKit/NetworkProcess/NetworkActivityTracker.h	2019-01-31 01:55:27 UTC (rev 240747)
@@ -64,7 +64,8 @@
         Failure,
     };
 
-    NetworkActivityTracker(Label, Domain = Domain::WebKit);
+    NetworkActivityTracker() = default;
+    explicit NetworkActivityTracker(Label, Domain = Domain::WebKit);
     ~NetworkActivityTracker();
 
     void setParent(NetworkActivityTracker&);
@@ -77,8 +78,8 @@
 
 private:
 #if HAVE(NW_ACTIVITY)
-    Domain m_domain;
-    Label m_label;
+    Domain m_domain { Domain::Invalid };
+    Label m_label { Label::Invalid };
     bool m_isCompleted { false };
     RetainPtr<nw_activity_t> m_networkActivity;
 #endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to