Creating a skeleton for Google Now for Chrome implementation.

chrome/browser/chrome_browser_main.cc

 // Copyright (c) 2012 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/chrome_browser_main.h"
 
 #if defined(TOOLKIT_GTK)
 #include <gtk/gtk.h>
 #endif
 
@@ skipping 66 matching lines @@
 #include "chrome/browser/search_engines/search_engine_type.h"
 #include "chrome/browser/search_engines/template_url.h"
 #include "chrome/browser/search_engines/template_url_prepopulate_data.h"
 #include "chrome/browser/search_engines/template_url_service.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/service/service_process_control.h"
 #include "chrome/browser/shell_integration.h"
 #include "chrome/browser/translate/translate_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/google_now/google_now_service_factory.h"
 #include "chrome/browser/ui/startup/default_browser_prompt.h"
 #include "chrome/browser/ui/startup/startup_browser_creator.h"
 #include "chrome/browser/ui/uma_browsing_activity_observer.h"
 #include "chrome/browser/ui/user_data_dir_dialog.h"
 #include "chrome/browser/ui/webui/chrome_url_data_manager_backend.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_result_codes.h"
 #include "chrome/common/chrome_switches.h"
@@ skipping 1302 matching lines @@
 
   // Create the instance of the cloud print proxy service so that it can launch
   // the service process if needed. This is needed because the service process
   // might have shutdown because an update was available.
   // TODO(torne): this should maybe be done with
   // ProfileKeyedServiceFactory::ServiceIsCreatedWithProfile() instead?
 #if !defined(OS_ANDROID)
   CloudPrintProxyServiceFactory::GetForProfile(profile_);
 #endif
 
+  // Create the instance of the Google Now service.
+#if !defined(OS_ANDROID) && !defined(OS_IOS)

[sky, 2012/12/06 16:40:54]

Would it be better to have a feature level ifdef, eg ENABLE_GOOGLE_NOW (similar
to ENABLE_LANGUAGE_DETECTION above).

[vadimt, 2012/12/06 20:43:16]

Done.

+  if (CommandLine::ForCurrentProcess()->HasSwitch(

[sky, 2012/12/06 16:40:54]

Do we really need to do this right at startup? Or said differently, won't doing
this now impact startup time?

[vadimt, 2012/12/06 20:43:16]

Well, it will instantiate several objects and post a task to start a geolocation
request. This is not too much, but probably, it's better to move this from the
startup. What place would you suggest?

[sky, 2012/12/06 21:07:07]

I don't know enough about this feature to suggest a good place. I would at least
wait until we've created all the browsers we need to.

[vadimt, 2012/12/07 02:41:23]

Moved past starting browser.

+      switches::kEnableGoogleNowIntegration)) {
+    GoogleNowServiceFactory::GetForProfile(profile_);
+  }
+#endif
+
   // Start watching all browser threads for responsiveness.
   ThreadWatcherList::StartWatchingAll(parsed_command_line());
 
 #if !defined(DISABLE_NACL)
   if (parsed_command_line().HasSwitch(switches::kPnaclDir)) {
     PathService::Override(chrome::DIR_PNACL_BASE,
                           parsed_command_line().GetSwitchValuePath(
                               switches::kPnaclDir));
   }
   NaClProcessHost::EarlyStartup();
@@ skipping 290 matching lines @@
     if (base::win::GetVersion() <= base::win::VERSION_XP)
       uma_name += "_XP";
 
     uma_name += "_PreRead_";
     uma_name += pre_read_percentage;
     AddPreReadHistogramTime(uma_name.c_str(), time);
   }
 #endif
 #endif
 }