Add default constructor for NetworkActivityTracker
authorkrollin@apple.com <krollin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 Jan 2019 01:55:27 +0000 (01:55 +0000)
committerkrollin@apple.com <krollin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 Jan 2019 01:55:27 +0000 (01:55 +0000)
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:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@240747 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/NetworkProcess/NetworkActivityTracker.h

index 18483e8..21e4bb2 100644 (file)
@@ -1,3 +1,20 @@
+2019-01-30  Keith Rollin  <krollin@apple.com>
+
+        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  <dsteffen@apple.com>
 
         <rdar://problem/29471922> Safari should switch from the legacy denap SPI to handling vouchers
index 88e3206..f2a0694 100644 (file)
@@ -64,7 +64,8 @@ public:
         Failure,
     };
 
-    NetworkActivityTracker(Label, Domain = Domain::WebKit);
+    NetworkActivityTracker() = default;
+    explicit NetworkActivityTracker(Label, Domain = Domain::WebKit);
     ~NetworkActivityTracker();
 
     void setParent(NetworkActivityTracker&);
@@ -77,8 +78,8 @@ public:
 
 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