Remove default constructors for QuicTime and QuicTime::Delta and used static Zero() methods instead…

net/quic/quic_time.cc

Index: net/quic/quic_time.cc
diff --git a/net/quic/quic_time.cc b/net/quic/quic_time.cc
index b0d6434ee3c3386249d6e02805c951ef16668f5b..68ea6aaa3d568d4943506f13ea0181ccf6744e9a 100644
--- a/net/quic/quic_time.cc
+++ b/net/quic/quic_time.cc
@@ -11,12 +11,12 @@ namespace net {
 // Highest number of microseconds that DateTimeOffset can hold.
 const int64 kQuicInfiniteTimeUs = GG_INT64_C(0x7fffffffffffffff) / 10;
 
-QuicTime::Delta::Delta()
-    : delta_(base::TimeDelta::FromMicroseconds(0)) {
+QuicTime::Delta::Delta(base::TimeDelta delta_)

[jar (doing other things), 2013/01/17 01:50:33]

nit: use an arg name without the trailing underscore.

[Ryan Hamilton, 2013/01/17 18:44:18]

Done.

+    : delta_(delta_) {
 }
 
-QuicTime::Delta::Delta(base::TimeDelta delta)
-    : delta_(delta) {
+QuicTime::Delta QuicTime::Delta::Zero() {
+  return QuicTime::Delta::FromMicroseconds(0);
 }
 
 QuicTime::Delta QuicTime::Delta::Infinite() {
@@ -61,8 +61,9 @@ QuicTime::Delta QuicTime::Delta::Subtract(const Delta& delta) const {
                                            delta.ToMicroseconds());
 }
 
-
-QuicTime::QuicTime() {
+// static
+QuicTime QuicTime::Zero() {
+  return QuicTime::FromMilliseconds(0);
 }
 
 QuicTime::QuicTime(base::TimeTicks ticks)