base::Version constructor from std::vector<uint32_t>

base/version.h

Index: base/version.h
diff --git a/base/version.h b/base/version.h
index 25b570a4e3f157636cd0e22f095ff368a56a30d9..3e2018e9b04a7ba8c107cca8941d4b0a65b03732 100644
--- a/base/version.h
+++ b/base/version.h
@@ -32,6 +32,8 @@ class BASE_EXPORT Version {
   // the outcome.
   explicit Version(const std::string& version_str);
 
+  explicit Version(std::vector<uint32_t>&& components);

[danakj, 2016/07/19 19:59:27]

this should come with an operator=.

This also deletes the implicit copy constructor. Is that intentional? 

[Alex Z., 2016/07/19 20:46:13]

Done.
A copy constructor is explicitly defined (and set to default) at line 26 and
base/version.cc:83.

[danakj, 2016/07/19 20:50:07]

Oh, there it is. Can you reorder this so that all the constructors are coming
before the destructor?

+
   // Returns true if the object contains a valid version number.
   bool IsValid() const;