Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4930)

Unified Diff: chrome_frame/npapi_url_request.cc

Issue 6166010: net: Remove typedef net::URLRequestStatus URLRequestStatus; (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome_frame/metrics_service.cc ('k') | chrome_frame/test/automation_client_mock.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/npapi_url_request.cc
diff --git a/chrome_frame/npapi_url_request.cc b/chrome_frame/npapi_url_request.cc
index 0a28cdde2a72f2ae7ea50e50a8d4044d773c1c62..26fa1c0e4cc5cd9498ae5c69a3e92eef548ad07c 100644
--- a/chrome_frame/npapi_url_request.cc
+++ b/chrome_frame/npapi_url_request.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
@@ -30,7 +30,7 @@ class NPAPIUrlRequest : public PluginUrlRequest {
virtual unsigned long API_CALL AddRef();
virtual unsigned long API_CALL Release();
- const URLRequestStatus& status() const {
+ const net::URLRequestStatus& status() const {
return status_;
}
@@ -43,7 +43,7 @@ class NPAPIUrlRequest : public PluginUrlRequest {
NPP instance_;
NPStream* stream_;
size_t pending_read_size_;
- URLRequestStatus status_;
+ net::URLRequestStatus status_;
base::PlatformThreadId thread_;
static int instance_count_;
@@ -55,7 +55,7 @@ int NPAPIUrlRequest::instance_count_ = 0;
NPAPIUrlRequest::NPAPIUrlRequest(NPP instance)
: ref_count_(0), instance_(instance), stream_(NULL),
pending_read_size_(0),
- status_(URLRequestStatus::FAILED, net::ERR_FAILED),
+ status_(net::URLRequestStatus::FAILED, net::ERR_FAILED),
thread_(base::PlatformThread::CurrentId()) {
DVLOG(1) << "Created request. Count: " << ++instance_count_;
}
@@ -112,7 +112,7 @@ bool NPAPIUrlRequest::Start() {
}
delegate_->OnResponseEnd(id(),
- URLRequestStatus(URLRequestStatus::FAILED, os_error));
+ net::URLRequestStatus(net::URLRequestStatus::FAILED, os_error));
return false;
}
@@ -123,7 +123,7 @@ void NPAPIUrlRequest::Stop() {
DVLOG(1) << "Finished request: Url - " << url()
<< " result: " << static_cast<int>(status_.status());
- status_.set_status(URLRequestStatus::CANCELED);
+ status_.set_status(net::URLRequestStatus::CANCELED);
if (stream_) {
npapi::DestroyStream(instance_, stream_, NPRES_USER_BREAK);
stream_ = NULL;
@@ -138,7 +138,7 @@ bool NPAPIUrlRequest::Read(int bytes_to_read) {
NPError NPAPIUrlRequest::OnStreamCreated(const char* mime_type,
NPStream* stream) {
stream_ = stream;
- status_.set_status(URLRequestStatus::IO_PENDING);
+ status_.set_status(net::URLRequestStatus::IO_PENDING);
// TODO(iyengar)
// Add support for passing persistent cookies and information about any URL
// redirects to Chrome.
@@ -154,19 +154,19 @@ NPError NPAPIUrlRequest::OnStreamDestroyed(NPReason reason) {
// instead. To prevent Chrome from receiving a notification of a failed
// network connection, when Chrome actually canceled the request, we ignore
// the status here.
- if (URLRequestStatus::CANCELED != status_.status()) {
+ if (net::URLRequestStatus::CANCELED != status_.status()) {
switch (reason) {
case NPRES_DONE:
- status_.set_status(URLRequestStatus::SUCCESS);
+ status_.set_status(net::URLRequestStatus::SUCCESS);
status_.set_os_error(0);
break;
case NPRES_USER_BREAK:
- status_.set_status(URLRequestStatus::CANCELED);
+ status_.set_status(net::URLRequestStatus::CANCELED);
status_.set_os_error(net::ERR_ABORTED);
break;
case NPRES_NETWORK_ERR:
default:
- status_.set_status(URLRequestStatus::FAILED);
+ status_.set_status(net::URLRequestStatus::FAILED);
status_.set_os_error(net::ERR_CONNECTION_CLOSED);
break;
}
@@ -334,8 +334,9 @@ void NPAPIUrlRequestManager::OnReadComplete(int request_id,
delegate_->OnReadComplete(request_id, data);
}
-void NPAPIUrlRequestManager::OnResponseEnd(int request_id,
- const URLRequestStatus& status) {
+void NPAPIUrlRequestManager::OnResponseEnd(
+ int request_id,
+ const net::URLRequestStatus& status) {
// Delete from map.
RequestMap::iterator it = request_map_.find(request_id);
DCHECK(request_map_.end() != it);
@@ -343,7 +344,7 @@ void NPAPIUrlRequestManager::OnResponseEnd(int request_id,
request_map_.erase(it);
// Inform delegate unless canceled.
- if (status.status() != URLRequestStatus::CANCELED)
+ if (status.status() != net::URLRequestStatus::CANCELED)
delegate_->OnResponseEnd(request_id, status);
}
@@ -363,7 +364,7 @@ NPError NPAPIUrlRequestManager::NewStream(NPMIMEType type,
// This stream is being constructed for a request that has already been
// canceled. Signal its immediate termination.
- if (URLRequestStatus::CANCELED == request->status().status()) {
+ if (net::URLRequestStatus::CANCELED == request->status().status()) {
return npapi::DestroyStream(request->instance(),
stream, NPRES_USER_BREAK);
}
« no previous file with comments | « chrome_frame/metrics_service.cc ('k') | chrome_frame/test/automation_client_mock.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698