Motown: Remove LPCM optimizations, fix prepare, add flush, add ActiveMultistreamSink model/stage

services/media/framework/safe_clone.h

+// Copyright 2016 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 SERVICES_MEDIA_FRAMEWORK_SAFE_CLONE_H_
+#define SERVICES_MEDIA_FRAMEWORK_SAFE_CLONE_H_
+
+#include <memory>
+#include <vector>
+
+namespace mojo {
+namespace media {
+
+template<typename T>
+std::unique_ptr<T> SafeClone(const std::unique_ptr<T>& t_ptr) {
+  return t_ptr ? t_ptr.get()->Clone() : nullptr;
+}
+
+template<typename T>
+std::unique_ptr<std::vector<std::unique_ptr<T>>> SafeClone(
+    const std::unique_ptr<std::vector<std::unique_ptr<T>>>& vec) {
+  if (vec == nullptr) {
+    return nullptr;
+  }
+
+  std::vector<std::unique_ptr<T>>* result =

[johngro, 2016/02/08 22:33:36]

why not just wrap result in a std::unique_ptr to begin with?

[dalesat, 2016/02/09 00:34:11]

This seemed simpler, but I don't really care one way or another.

+      new std::vector<std::unique_ptr<T>>(vec->size());
+  for (const std::unique_ptr<T>& t_ptr : *vec.get()) {
+    result->push_back(SafeClone(t_ptr));
+  }
+
+  return std::unique_ptr<std::vector<std::unique_ptr<T>>>(result);
+}
+
+}  // namespace media
+}  // namespace mojo
+
+#endif // SERVICES_MEDIA_FRAMEWORK_SAFE_CLONE_H_