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

Unified Diff: mojo/runner/about_fetcher.cc

Issue 1342503003: Move fetching logic out of ApplicationManager, eliminate url mappings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 5 years, 3 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 | « mojo/runner/about_fetcher.h ('k') | mojo/runner/about_fetcher_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/runner/about_fetcher.cc
diff --git a/mojo/runner/about_fetcher.cc b/mojo/runner/about_fetcher.cc
deleted file mode 100644
index e521b80b019c390190e6dc02094bc0262da6f1cd..0000000000000000000000000000000000000000
--- a/mojo/runner/about_fetcher.cc
+++ /dev/null
@@ -1,111 +0,0 @@
-// 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 "mojo/runner/about_fetcher.h"
-
-#include "base/bind.h"
-#include "base/files/file_path.h"
-#include "base/logging.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/message_loop/message_loop.h"
-
-namespace mojo {
-namespace runner {
-namespace {
-
-void RunFetcherCallback(const shell::Fetcher::FetchCallback& callback,
- scoped_ptr<shell::Fetcher> fetcher,
- bool success) {
- callback.Run(success ? fetcher.Pass() : nullptr);
-}
-
-} // namespace
-
-const char AboutFetcher::kAboutScheme[] = "about";
-const char AboutFetcher::kAboutBlankURL[] = "about:blank";
-
-// static
-void AboutFetcher::Start(const GURL& url,
- const FetchCallback& loader_callback) {
- // The object manages its own lifespan.
- new AboutFetcher(url, loader_callback);
-}
-
-AboutFetcher::AboutFetcher(const GURL& url,
- const FetchCallback& loader_callback)
- : Fetcher(loader_callback), url_(url) {
- BuildResponse();
-}
-
-AboutFetcher::~AboutFetcher() {}
-
-void AboutFetcher::BuildResponse() {
- if (url_ != GURL(kAboutBlankURL)) {
- PostToRunCallback(false);
- return;
- }
-
- response_ = URLResponse::New();
- response_->url = kAboutBlankURL;
- response_->status_code = 200;
- response_->mime_type = "text/html";
- PostToRunCallback(true);
-}
-
-void AboutFetcher::PostToRunCallback(bool success) {
- // Also pass |this| on failure, so that we won't destroy the object while the
- // constructor is still on the call stack.
- base::MessageLoop::current()->PostTask(
- FROM_HERE,
- base::Bind(RunFetcherCallback, loader_callback_,
- base::Passed(make_scoped_ptr<Fetcher>(this)), success));
-}
-
-const GURL& AboutFetcher::GetURL() const {
- return url_;
-}
-
-GURL AboutFetcher::GetRedirectURL() const {
- return GURL::EmptyGURL();
-}
-
-GURL AboutFetcher::GetRedirectReferer() const {
- return GURL::EmptyGURL();
-}
-
-URLResponsePtr AboutFetcher::AsURLResponse(base::TaskRunner* task_runner,
- uint32_t skip) {
- DCHECK(response_);
-
- // Ignore |skip| because the only URL handled currently is "about:blank" which
- // doesn't have a body.
- DCHECK(!response_->body.is_valid());
-
- return response_.Pass();
-}
-
-void AboutFetcher::AsPath(
- base::TaskRunner* task_runner,
- base::Callback<void(const base::FilePath&, bool)> callback) {
- NOTIMPLEMENTED();
- base::MessageLoop::current()->PostTask(
- FROM_HERE, base::Bind(callback, base::FilePath(), false));
-}
-
-std::string AboutFetcher::MimeType() {
- DCHECK(response_);
- return response_->mime_type;
-}
-
-bool AboutFetcher::HasMojoMagic() {
- return false;
-}
-
-bool AboutFetcher::PeekFirstLine(std::string* line) {
- // The only URL handled currently is "about:blank" which doesn't have a body.
- return false;
-}
-
-} // namespace runner
-} // namespace mojo
« no previous file with comments | « mojo/runner/about_fetcher.h ('k') | mojo/runner/about_fetcher_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698