Motown in-proc streaming framework used to implement media services.

services/media/framework/parts/reader.cc

Index: services/media/framework/parts/reader.cc
diff --git a/services/media/framework/parts/reader.cc b/services/media/framework/parts/reader.cc
new file mode 100644
index 0000000000000000000000000000000000000000..24ac7a1093a0036c6bd98833d9da11e3d7933709
--- /dev/null
+++ b/services/media/framework/parts/reader.cc
@@ -0,0 +1,37 @@
+// 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.
+
+#include "base/logging.h"
+#include "services/media/framework/parts/file_reader.h"
+#include "services/media/framework/parts/reader.h"
+#include "url/gurl.h"
+
+namespace mojo {
+namespace media {
+
+Result Reader::Create(const GURL& gurl, ReaderPtr* reader_out) {
+  if (!gurl.is_valid()) {
+    return Result::kInvalidArgument;
+  }
+
+  ReaderPtr reader = nullptr;
+  if (gurl.SchemeIsFile()) {
+    reader = FileReader::New();
+  }
+  // TODO(dalesat): More schemes.

[jeffbrown, 2016/02/02 05:35:48]

Honestly this code doesn't belong in your framework at all.  Resolving content
by URL is something that should be done via a service.

[dalesat, 2016/02/02 21:46:40]

I agree that *most* of the resolution should be done via a service. Are file I/O
and network I/O liable to use the same interface? If not, we'll need to make
that distinction at least. The rubber has to hit the road somewhere.

+
+  if (reader == nullptr) {
+    return Result::kUnsupportedOperation;
+  }
+
+  Result result = reader->Init(gurl);
+  if (result == Result::kOk) {
+    *reader_out = reader;
+  }
+
+  return result;
+}
+
+} // namespace media
+} // namespace mojo