mojo: fix little bug in DynamicServiceLoader/Runner

mojo/shell/dynamic_service_loader.cc

 // Copyright 2014 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/shell/dynamic_service_loader.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
@@ skipping 49 matching lines @@
   explicit LocalLoader(scoped_ptr<DynamicServiceRunner> runner)
       : Loader(runner.Pass()) {
   }
 
   virtual void Start(const GURL& url,
                      ScopedMessagePipeHandle service_handle,
                      Context* context) OVERRIDE {
     base::FilePath path;
     net::FileURLToFilePath(url, &path);
 
-    // TODO(darin): Check if the given file path exists.
-
     // Complete asynchronously for consistency with NetworkServiceLoader.
     base::MessageLoop::current()->PostTask(
         FROM_HERE,
         base::Bind(&Loader::StartService,
                    base::Unretained(this),
                    path,
                    base::Passed(&service_handle),
-                   true));
+                   base::PathExists(path)));
   }
 };
 
 // For loading services via the network stack.
 class NetworkLoader : public Loader {
  public:
   explicit NetworkLoader(scoped_ptr<DynamicServiceRunner> runner,
                          NetworkService* network_service)
       : Loader(runner.Pass()) {
     network_service->CreateURLLoader(Get(&url_loader_));
@@ skipping 88 matching lines @@
 }
 
 void DynamicServiceLoader::OnServiceError(ServiceManager* manager,
                                           const GURL& url) {
   // TODO(darin): What should we do about service errors? This implies that
   // the app closed its handle to the service manager. Maybe we don't care?
 }
 
 }  // namespace shell
 }  // namespace mojo