Introduce FastShowPickler which will be used for app list cold start.

chrome/chrome_browser_ui.gypi

 # Copyright 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.
 
 {
   'targets': [
     {
       'target_name': 'browser_ui',
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
@@ skipping 90 matching lines @@
         'browser/ui/android/tab_model/tab_model.h',
         'browser/ui/android/tab_model/tab_model_list.cc',
         'browser/ui/android/tab_model/tab_model_list.h',
         'browser/ui/android/tab_restore_service_delegate_android.cc',
         'browser/ui/android/validation_message_bubble_android.h',
         'browser/ui/android/validation_message_bubble_android.cc',
         'browser/ui/android/website_settings_popup_android.cc',
         'browser/ui/android/website_settings_popup_android.h',
         'browser/ui/android/window_android_helper.cc',
         'browser/ui/android/window_android_helper.h',
+        'browser/ui/app_list/fast_show_pickler.cc',
+        'browser/ui/app_list/fast_show_pickler.h',
         'browser/ui/app_list/app_context_menu.cc',
         'browser/ui/app_list/app_context_menu.h',
         'browser/ui/app_list/app_context_menu_delegate.h',
         'browser/ui/app_list/app_list_controller_delegate.cc',
         'browser/ui/app_list/app_list_controller_delegate.h',
         'browser/ui/app_list/app_list_service.cc',
         'browser/ui/app_list/app_list_service.h',
         'browser/ui/app_list/app_list_service_ash.h',
         'browser/ui/app_list/app_list_service_disabled.cc',
         'browser/ui/app_list/app_list_service_impl.cc',
@@ skipping 2986 matching lines @@
         ['enable_mdns==1', {
           'sources': [
             'browser/ui/webui/local_discovery/local_discovery_ui.cc',
             'browser/ui/webui/local_discovery/local_discovery_ui.h',
             'browser/ui/webui/local_discovery/local_discovery_ui_handler.cc',
             'browser/ui/webui/local_discovery/local_discovery_ui_handler.h',
           ],
         }],
       ],
     },
+    {
+      # Unit tests for chrome app list, not run on any bots, this is for faster
+      # compile/link/test cycles during development.
+      'target_name': 'chrome_app_list_unit_tests',
+      'type': '<(gtest_target_type)',
+      'dependencies': [
+        '../skia/skia.gyp:skia',
+        '../testing/gtest.gyp:gtest',
+        '../ui/app_list/app_list.gyp:app_list',
+        'apps',
+        'browser_ui',
+      ],
+      'sources': [
+        'browser/ui/app_list/chrome_app_list_unit_tests.cc',

[tapted, 2013/08/30 01:13:35]

Drive-by: should this go in a test subfolder. I.e.
chrome/browser/ui/app_list/test/ ?

And I think it's more common for there to be no underscore in unit_tests. i.e.
target/file = chrome_app_list_unittests  (to match e.g. ui_unittests,
base_unittests, etc.).

Poking around... it may even be convention to have the source file always called
'run_all_unittests.cc', with the target/path deciding what it's actually for..
But these all look the following, so maybe work different (I haven't really
played that much with gtest..)


#include "base/test/test_suite.h"

int main(int argc, char** argv) {
  return base::TestSuite(argc, argv).Run();
}

[koz (OOO until 15th September), 2013/08/30 05:54:23]

Ooh, thanks for the suggestion Trent. It seems that such a file isn't even
needed if we depend on base/base.gyp:run_all_unittests which will link in a
main()/test runner for you.

+        'browser/ui/app_list/fast_show_pickler_unittest.cc',
+      ],
+    },
   ],
 }