Implement new strategy for default apps

chrome/browser/extensions/crx_installer.cc

 // Copyright (c) 2010 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 "chrome/browser/extensions/crx_installer.h"
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
@@ skipping 172 matching lines @@
         original_mime_type_ != Extension::kMimeType) {
       *error = base::StringPrintf(
           "Apps must be served with content type %s.",
           Extension::kMimeType);
       return false;
     }
 
     // For self-hosted apps, verify that the entire extent is on the same
     // host (or a subdomain of the host) the download happened from.  There's
     // no way for us to verify that the app controls any other hosts.
-    if (!is_gallery_install_) {
+    if (!is_gallery_install_ && client_) {
       URLPattern pattern(UserScript::kValidUserScriptSchemes);
       pattern.set_host(original_url_.host());
       pattern.set_match_subdomains(true);
 
       ExtensionExtent::PatternList patterns =
           extension_->web_extent().patterns();
       for (size_t i = 0; i < patterns.size(); ++i) {
         if (!pattern.MatchesHost(patterns[i].host())) {
           *error = base::StringPrintf(
               "Apps must be served from the host that they affect.");
@@ skipping 193 matching lines @@
     client_->OnInstallSuccess(extension_.get());
 
   // Tell the frontend about the installation and hand off ownership of
   // extension_ to it.
   frontend_->OnExtensionInstalled(extension_.release(),
                                   allow_privilege_increase_);
 
   // We're done. We don't post any more tasks to ourselves so we are deleted
   // soon.
 }