Change who allocated the MediaPipe's shared buffer.

services/media/audio/audio_track_impl.cc

 // Copyright 2015 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.
 
 #include <algorithm>
 #include <limits>
 
 #include "base/logging.h"
 #include "mojo/services/media/common/cpp/linear_transform.h"
 #include "services/media/audio/audio_output_manager.h"
@@ skipping 207 matching lines @@
       bytes_per_frame_ = 4;
       break;
 
     default:
       DCHECK(false);
       bytes_per_frame_ = 2;
       break;
   }
   bytes_per_frame_ *= cfg->channels;
 
-  // Overflow trying to convert from frames to bytes?
-  uint64_t requested_frames = configuration->max_frames;
-  if (requested_frames >
-     (std::numeric_limits<size_t>::max() / bytes_per_frame_)) {
-    LOG(ERROR) << "Insufficient resources to create "
-               << requested_frames << " frame audio buffer.";
+  // Bind our pipe to the interface request.
+  if (pipe_.Init(req.Pass()) != MOJO_RESULT_OK) {
+    LOG(ERROR) << "Failed to media pipe to interface request.";
     Shutdown();
     return;
   }
-
-  size_t requested_bytes = (requested_frames * bytes_per_frame_);
-
-  // Attempt to initialize our shared buffer and bind it to our interface
-  // request.
-  if (pipe_.Init(req.Pass(), requested_bytes) != MOJO_RESULT_OK) {
-    LOG(ERROR) << "Insufficient resources to create "
-               << requested_frames << " frame audio buffer.";
-    Shutdown();
-    return;
-  }
 
   // Stash our configuration.
   format_ = cfg.Pass();
 
   // Have the audio output manager initialize our set of outputs.  Note; there
   // is currently no need for a lock here.  Methods called from our user-facing
   // interfaces are seriailzed by nature of the mojo framework, and none of the
   // output manager's threads should ever need to manipulate the set.  Cleanup
   // of outputs which have gone away is currently handled in a lazy fashion when
   // the track fails to promote its weak reference during an operation involving
@@ skipping 69 matching lines @@
     DCHECK(output);
     output->FlushPendingQueue();
   }
   cbk.Run();
   return true;
 }
 
 }  // namespace audio
 }  // namespace media
 }  // namespace mojo