Mandoline: let html_viewer handle more contents

mojo/fetcher/about_fetcher.cc

 // 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/fetcher/about_fetcher.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 23 matching lines @@
 
 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 = url_.spec();
 
-  response_ = URLResponse::New();
-  response_->url = kAboutBlankURL;
-  response_->status_code = 200;
+  // about: URLs other than about:blank are not supported yet.
+  //
+  // TODO(yzshen): crbug.com/516494 Eventually we need a general solution to
+  // generate error page for network errors/unrecognized app format/etc.
+  response_->status_code = (url_ == GURL(kAboutBlankURL)) ? 200 : 404;
+
   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_,
@@ skipping 40 matching lines @@
   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 fetcher
 }  // namespace mojo