Revert 95009. Causing pyauto NTP test failures on all platforms.

chrome/browser/automation/automation_provider_observers.cc

 // Copyright (c) 2011 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/automation/automation_provider_observers.h"
 
 #include <deque>
 #include <string>
 #include <vector>
 
@@ skipping 1963 matching lines @@
 // of the created vector.
 std::vector<DictionaryValue*>* GetAppInfoFromExtensions(
     const ExtensionList* extensions,
     ExtensionService* ext_service) {
   std::vector<DictionaryValue*>* apps_list =
       new std::vector<DictionaryValue*>();
   for (ExtensionList::const_iterator ext = extensions->begin();
        ext != extensions->end(); ++ext) {
     // Only return information about extensions that are actually apps.
     if ((*ext)->is_app()) {
-      DictionaryValue* app_info =
-          AppLauncherHandler::CreateAppInfo(*ext, NULL, ext_service);
-      if (!app_info)
-        continue;
+      DictionaryValue* app_info = new DictionaryValue();
+      AppLauncherHandler::CreateAppInfo(*ext, NULL, ext_service, app_info);
       app_info->SetBoolean("is_component_extension",
                            (*ext)->location() == Extension::COMPONENT);
 
       // Convert the launch_type integer into a more descriptive string.
       int launch_type;
       const char* kLaunchType = "launch_type";
       if (!app_info->GetInteger(kLaunchType, &launch_type)) {
         NOTREACHED() << "Can't get integer from key " << kLaunchType;
         continue;
       }
@@ skipping 654 matching lines @@
 void DragTargetDropAckNotificationObserver::Observe(
     int type,
     const NotificationSource& source,
     const NotificationDetails& details) {
   if (automation_) {
     AutomationJSONReply(automation_,
                         reply_message_.release()).SendSuccess(NULL);
   }
   delete this;
 }