aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristopher Wiley <wiley@google.com>2016-04-26 16:17:51 +0000
committerandroid-build-merger <android-build-merger@google.com>2016-04-26 16:17:51 +0000
commit86e480f5c234d7a1597bc7bc787ffc455ee8aa70 (patch)
treee0bdc97bbdcab1b4b749ec61b7a557f4b871cf09
parent7c79b9745807bfbe6e0d7b3f9ebc5e74e6fbaa9b (diff)
parent87136aa7a233fbdbaeebf291c9598876adaa11b2 (diff)
downloadwebservd-86e480f5c234d7a1597bc7bc787ffc455ee8aa70.tar.gz
Replace ScopedFd with unique_fd am: 7fb886f
am: 87136aa * commit '87136aa7a233fbdbaeebf291c9598876adaa11b2': Replace ScopedFd with unique_fd Change-Id: I3df1d0f8ab413c067a144ca769f014b9e347362b
-rw-r--r--libwebserv/binder_request_handler.cc2
-rw-r--r--libwebserv/binder_request_impl.cc2
-rw-r--r--libwebserv/binder_request_impl.h6
-rw-r--r--libwebserv/binder_response.cc2
-rw-r--r--webservd/binder_request.cc6
-rw-r--r--webservd/binder_request.h8
6 files changed, 13 insertions, 13 deletions
diff --git a/libwebserv/binder_request_handler.cc b/libwebserv/binder_request_handler.cc
index 3891135..9dd4325 100644
--- a/libwebserv/binder_request_handler.cc
+++ b/libwebserv/binder_request_handler.cc
@@ -33,7 +33,7 @@ Status BinderRequestHandler::ProcessRequest(const sp<IHttpRequest>& request) {
vector<string> headers;
vector<string> query_params;
vector<string> post_params;
- ScopedFd body;
+ android::base::unique_fd body;
// TODO(b/27407721): Fewer binder calls here
status = request->GetUrl(&url);
diff --git a/libwebserv/binder_request_impl.cc b/libwebserv/binder_request_impl.cc
index 2a81e29..e93ce27 100644
--- a/libwebserv/binder_request_impl.cc
+++ b/libwebserv/binder_request_impl.cc
@@ -24,7 +24,7 @@ BinderRequestImpl::BinderRequestImpl(const string& url,
const vector<string>& headers,
const vector<string>& query_params,
const vector<string>& post_params,
- ScopedFd&& body)
+ android::base::unique_fd&& body)
: Request(url, method),
body_(std::move(body)) {
CHECK(!(headers.size() % 2));
diff --git a/libwebserv/binder_request_impl.h b/libwebserv/binder_request_impl.h
index f8e2a0f..eae2b78 100644
--- a/libwebserv/binder_request_impl.h
+++ b/libwebserv/binder_request_impl.h
@@ -20,7 +20,7 @@
#include <brillo/streams/file_stream.h>
-#include <nativehelper/ScopedFd.h>
+#include <android-base/unique_fd.h>
#include "libwebserv/request.h"
@@ -33,12 +33,12 @@ class BinderRequestImpl : public Request {
const std::vector<std::string>& headers,
const std::vector<std::string>& query_params,
const std::vector<std::string>& post_params,
- ScopedFd&& body);
+ android::base::unique_fd&& body);
brillo::StreamPtr GetDataStream() override;
private:
- ScopedFd body_;
+ android::base::unique_fd body_;
DISALLOW_COPY_AND_ASSIGN(BinderRequestImpl);
};
diff --git a/libwebserv/binder_response.cc b/libwebserv/binder_response.cc
index ddcffab..4f497f8 100644
--- a/libwebserv/binder_response.cc
+++ b/libwebserv/binder_response.cc
@@ -44,7 +44,7 @@ void BinderResponse::Reply(int status_code,
const std::string& mime_type) {
vector<string> headers;
AddHeader(brillo::http::response_header::kContentType, mime_type);
- ScopedFd data_out;
+ android::base::unique_fd data_out;
for (const auto& header : headers_) {
headers.emplace_back(header.first);
diff --git a/webservd/binder_request.cc b/webservd/binder_request.cc
index 96c0a86..4b7d1ea 100644
--- a/webservd/binder_request.cc
+++ b/webservd/binder_request.cc
@@ -33,7 +33,7 @@ namespace android {
namespace webservd {
namespace {
-Status DupToScopedFd(int fd, ScopedFd* target) {
+Status DupToScopedFd(int fd, android::base::unique_fd* target) {
do {
target->reset(dup(fd));
@@ -51,7 +51,7 @@ Status DupToScopedFd(int fd, ScopedFd* target) {
} // namespace
-Status HttpRequest::GetBody(ScopedFd* body) {
+Status HttpRequest::GetBody(android::base::unique_fd* body) {
return DupToScopedFd(request_->GetBodyDataFileDescriptor(), body);
}
@@ -93,7 +93,7 @@ Status HttpRequest::GetHeaders(vector<string>* headers) {
Status HttpRequest::Respond(int32_t status_code,
const vector<string>& response_headers,
int32_t data_size,
- ScopedFd* response_stream) {
+ android::base::unique_fd* response_stream) {
if (response_headers.size() % 2) {
return Status::fromExceptionCode(Status::EX_ILLEGAL_ARGUMENT,
String8("Header array contained unmatched key."));
diff --git a/webservd/binder_request.h b/webservd/binder_request.h
index 1e8e74a..9fe7f28 100644
--- a/webservd/binder_request.h
+++ b/webservd/binder_request.h
@@ -19,7 +19,7 @@
#include <string>
#include <vector>
-#include <nativehelper/ScopedFd.h>
+#include <android-base/unique_fd.h>
#include "android/webservd/BnHttpRequest.h"
@@ -33,7 +33,7 @@ class HttpRequest final : public android::webservd::BnHttpRequest {
public:
explicit HttpRequest(::webservd::Request* request) : request_(request) {}
- android::binder::Status GetBody(ScopedFd* body) override;
+ android::binder::Status GetBody(android::base::unique_fd* body) override;
android::binder::Status GetUrl(std::string* url) override;
android::binder::Status GetMethod(std::string* method) override;
android::binder::Status GetQueryParams(std::vector<std::string>* params)
@@ -46,11 +46,11 @@ class HttpRequest final : public android::webservd::BnHttpRequest {
int32_t status_code,
const std::vector<std::string>& response_headers,
int32_t data_size,
- ScopedFd* response_stream) override;
+ android::base::unique_fd* response_stream) override;
private:
::webservd::Request* request_;
- ScopedFd body_;
+ android::base::unique_fd body_;
DISALLOW_COPY_AND_ASSIGN(HttpRequest);
};