net: Return size of upload as well as position from HttpTransaction::GetUploadProgress()

net/base/upload_progress.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NET_BASE_UPLOAD_PROGRESS_H_
+#define NET_BASE_UPLOAD_PROGRESS_H_
+
+#include "base/basictypes.h"
+
+namespace net {
+
+struct UploadProgress {
+  UploadProgress() : size(0), position(0) {}
+  UploadProgress(uint64 position, uint64 size)
+      : size(size), position(position) {}
+  uint64 size;

[darin (slow to review), 2012/08/29 22:05:35]

do these need to be public variables?  do they need to be mutable?  they do not
appear to need to be mutable for any of the code in this CL.

[hashimoto, 2012/08/29 22:27:08]

I think UploadProgress is simple enough to be a struct, not a class.
Made a new change in which UploadProgress is a class.
Is this what you mean?

+  uint64 position;
+};
+
+}  // namespace net
+
+#endif  // NET_BASE_UPLOAD_PROGRESS_H_