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

Unified Diff: content/browser/loader/certificate_resource_handler.cc

Issue 1423663012: Removing x-x509-user-cert mime handler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Using MimeType for handler. Created 5 years, 1 month 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
Index: content/browser/loader/certificate_resource_handler.cc
diff --git a/content/browser/loader/certificate_resource_handler.cc b/content/browser/loader/certificate_resource_handler.cc
deleted file mode 100644
index fd49181f8daabc464260813a3787fafdd3178240..0000000000000000000000000000000000000000
--- a/content/browser/loader/certificate_resource_handler.cc
+++ /dev/null
@@ -1,111 +0,0 @@
-// Copyright (c) 2012 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/loader/certificate_resource_handler.h"
-
-#include <limits.h>
-
-#include "components/mime_util/mime_util.h"
-#include "content/browser/loader/resource_request_info_impl.h"
-#include "content/public/browser/content_browser_client.h"
-#include "content/public/common/resource_response.h"
-#include "net/base/io_buffer.h"
-#include "net/url_request/url_request.h"
-#include "net/url_request/url_request_status.h"
-
-namespace content {
-
-CertificateResourceHandler::CertificateResourceHandler(net::URLRequest* request)
- : ResourceHandler(request),
- buffer_(new net::GrowableIOBuffer),
- cert_type_(net::CERTIFICATE_MIME_TYPE_UNKNOWN) {
-}
-
-CertificateResourceHandler::~CertificateResourceHandler() {
-}
-
-bool CertificateResourceHandler::OnRequestRedirected(
- const net::RedirectInfo& redirect_info,
- ResourceResponse* resp,
- bool* defer) {
- return true;
-}
-
-bool CertificateResourceHandler::OnResponseStarted(ResourceResponse* resp,
- bool* defer) {
- cert_type_ =
- mime_util::GetCertificateMimeTypeForMimeType(resp->head.mime_type);
- return cert_type_ != net::CERTIFICATE_MIME_TYPE_UNKNOWN;
-}
-
-bool CertificateResourceHandler::OnWillStart(const GURL& url, bool* defer) {
- return true;
-}
-
-bool CertificateResourceHandler::OnBeforeNetworkStart(const GURL& url,
- bool* defer) {
- return true;
-}
-
-bool CertificateResourceHandler::OnWillRead(scoped_refptr<net::IOBuffer>* buf,
- int* buf_size,
- int min_size) {
- static const int kInitialBufferSizeInBytes = 32768;
- static const int kMaxCertificateSizeInBytes = 1024 * 1024;
-
- // TODO(gauravsh): Should we use 'min_size' here?
- DCHECK(buf);
- DCHECK(buf_size);
-
- if (buffer_->capacity() == 0) {
- buffer_->SetCapacity(kInitialBufferSizeInBytes);
- } else if (buffer_->RemainingCapacity() == 0) {
- int capacity = buffer_->capacity();
- if (capacity >= kMaxCertificateSizeInBytes)
- return false;
- static_assert(kMaxCertificateSizeInBytes < INT_MAX / 2,
- "The size limit ensures the capacity remains in bounds.");
- capacity *= 2;
- if (capacity > kMaxCertificateSizeInBytes)
- capacity = kMaxCertificateSizeInBytes;
- buffer_->SetCapacity(capacity);
- }
-
- *buf = buffer_.get();
- *buf_size = buffer_->RemainingCapacity();
-
- return true;
-}
-
-bool CertificateResourceHandler::OnReadCompleted(int bytes_read, bool* defer) {
- DCHECK_LE(0, bytes_read);
- DCHECK_LE(bytes_read, buffer_->RemainingCapacity());
- if (!bytes_read)
- return true;
-
- buffer_->set_offset(buffer_->offset() + bytes_read);
- return true;
-}
-
-void CertificateResourceHandler::OnResponseCompleted(
- const net::URLRequestStatus& urs,
- const std::string& sec_info,
- bool* defer) {
- if (urs.status() != net::URLRequestStatus::SUCCESS)
- return;
-
- // Note that it's up to the browser to verify that the certificate
- // data is well-formed.
- const ResourceRequestInfo* info = GetRequestInfo();
- GetContentClient()->browser()->AddCertificate(
- cert_type_, buffer_->StartOfBuffer(),
- static_cast<size_t>(buffer_->offset()), info->GetChildID(),
- info->GetRenderFrameID());
-}
-
-void CertificateResourceHandler::OnDataDownloaded(int bytes_downloaded) {
- NOTREACHED();
-}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698