Move MediaDrmBridge provision communication to native side.

content/browser/media/android/url_provision_fetcher.cc

Index: content/browser/media/android/url_provision_fetcher.cc
diff --git a/content/browser/media/android/url_provision_fetcher.cc b/content/browser/media/android/url_provision_fetcher.cc
new file mode 100644
index 0000000000000000000000000000000000000000..c5654eac495ef7ca11c222e4b8adc86f490d51fa
--- /dev/null
+++ b/content/browser/media/android/url_provision_fetcher.cc
@@ -0,0 +1,76 @@
+// 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 "content/browser/media/android/url_provision_fetcher.h"
+
+#include <sstream>
+
+#include "content/public/browser/android/cdm_provision_fetcher.h"
+#include "media/base/bind_to_current_loop.h"
+#include "net/url_request/url_fetcher.h"
+
+namespace content {
+
+using net::URLFetcher;
+
+URLProvisionFetcher::URLProvisionFetcher(net::URLRequestContextGetter* context)
+    : url_request_context_(context) {}
+
+URLProvisionFetcher::~URLProvisionFetcher() {}
+
+void URLProvisionFetcher::Retrieve(
+    const std::string& default_url,
+    const std::string& request_data,
+    const media::ProvisionFetcher::ResponseCB& response_cb) {
+  response_cb_ = media::BindToCurrentLoop(response_cb);

[xhwang, 2015/11/11 09:53:21]

Can response_cb_ be fired on a different thread? If not, we don't need
BindToCurrentLoop... Otherwise, we should add a comment.

[Tima Vaisburd, 2015/11/11 23:26:35]

Yes, this was unaswered in PS2. The URLFetcher implementation seems to guarantee
that OnURLFetchComplete is called on the same thread that called Start(): 
https://code.google.com/p/chromium/codesearch#chromium/src/net/url_request/ur...

It also posts a task, so the call will be async.
I put this into comment and removed BindToCurentLoop, however, should we rely on
the current net implementation in this respect? 

[xhwang, 2015/11/12 22:27:03]

This should really be part of the URLFetcher's public interface instead of
implementation detail...

+
+  std::ostringstream os;
+  os << default_url << "&signedRequest=" << request_data;
+  DVLOG(1) << __FUNCTION__ << ": request:" << os.str();
+
+  DCHECK(!request_);
+  request_ = URLFetcher::Create(GURL(os.str()), URLFetcher::POST, this);
+
+  // SetUploadData is mandatory even if we are not uploading anything.
+  request_->SetUploadData("", "");
+  request_->AddExtraRequestHeader("User-Agent: Widevine CDM v1.0");
+  request_->AddExtraRequestHeader("Content-Type: application/json");
+
+  DCHECK(url_request_context_);
+  request_->SetRequestContext(url_request_context_);
+
+  request_->Start();
+}
+
+void URLProvisionFetcher::OnURLFetchComplete(const net::URLFetcher* source) {
+  DCHECK(source);
+
+  int response_code = source->GetResponseCode();
+  DVLOG(1) << __FUNCTION__ << ": response code:" << source->GetResponseCode();
+
+  std::string response;
+  bool success = false;
+  if (response_code == 200) {
+    success = source->GetResponseAsString(&response);
+    (void)(success);

[xhwang, 2015/11/11 09:53:21]

not needed?

[Tima Vaisburd, 2015/11/11 23:26:35]

Sorry, it was bad thinking about unused variable. Removed. 

+    DVLOG_IF(1, !success) << __FUNCTION__ << ": GetResponseAsString() failed";
+  } else {
+    DVLOG(1) << "CDM provision: server returned error code " << response_code;
+  }
+
+  request_.reset();
+
+  DCHECK(!response_cb_.is_null());
+  response_cb_.Run(success, response);
+}
+
+// CDMProvisionFetcher implementation.
+
+// static
+scoped_ptr<media::ProvisionFetcher> CDMProvisionFetcher::CreateWithURLContext(
+    net::URLRequestContextGetter* context) {
+  return scoped_ptr<media::ProvisionFetcher>(new URLProvisionFetcher(context));
+}
+
+}  // namespace content