crostini: fix LinuxPackageInfo constructors

The explicit destructor is pessimising performance by preventing the move constructor from being generated.

Change-Id: Ia595d72a631decd522e22189fe42ec235fd7e381
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2029402
Commit-Queue: Julian Watson <juwa@google.com>
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Auto-Submit: Julian Watson <juwa@google.com>
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#736764}

chrome/browser/chromeos/crostini/crostini_manager.cc

@@ -632,7 +632,11 @@ void CrostiniManager::AddRunningVmForTesting(std::string vm_name) {
 }
 
 LinuxPackageInfo::LinuxPackageInfo() = default;
+LinuxPackageInfo::LinuxPackageInfo(LinuxPackageInfo&&) = default;
 LinuxPackageInfo::LinuxPackageInfo(const LinuxPackageInfo&) = default;
+LinuxPackageInfo& LinuxPackageInfo::operator=(LinuxPackageInfo&&) = default;
+LinuxPackageInfo& LinuxPackageInfo::operator=(const LinuxPackageInfo&) =
+    default;
 LinuxPackageInfo::~LinuxPackageInfo() = default;
 
 ContainerInfo::ContainerInfo(std::string container_name,

chrome/browser/chromeos/crostini/crostini_simple_types.h

@@ -161,7 +161,10 @@ struct Icon {
 
 struct LinuxPackageInfo {
   LinuxPackageInfo();
+  LinuxPackageInfo(LinuxPackageInfo&&);
   LinuxPackageInfo(const LinuxPackageInfo&);
+  LinuxPackageInfo& operator=(LinuxPackageInfo&&);
+  LinuxPackageInfo& operator=(const LinuxPackageInfo&);
   ~LinuxPackageInfo();
 
   bool success;