Delay updating jumplist to avoid blocking the file thread at start-up

chrome/browser/ui/views/frame/browser_view.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/timer.h"
 #include "build/build_config.h"
 #include "chrome/browser/devtools/devtools_window.h"
 #include "chrome/browser/infobars/infobar_container.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
 #include "chrome/browser/ui/views/frame/browser_frame.h"
+#include "chrome/browser/ui/views/load_complete_listener.h"
 #include "chrome/browser/ui/views/unhandled_keyboard_event_handler.h"
 #include "ui/base/accelerators/accelerator.h"
 #include "ui/base/models/simple_menu_model.h"
 #include "ui/gfx/native_widget_types.h"
 #include "ui/gfx/sys_color_change_listener.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/single_split_view_listener.h"
 #include "ui/views/widget/widget_delegate.h"
 #include "ui/views/widget/widget_observer.h"
 #include "ui/views/window/client_view.h"
@@ skipping 56 matching lines @@
 //
 class BrowserView : public BrowserWindow,
                     public BrowserWindowTesting,
                     public TabStripModelObserver,
                     public ui::AcceleratorProvider,
                     public views::WidgetDelegate,
                     public views::WidgetObserver,
                     public views::ClientView,
                     public InfoBarContainer::Delegate,
                     public views::SingleSplitViewListener,
-                    public gfx::SysColorChangeListener {
+                    public gfx::SysColorChangeListener,
+                    public LoadCompleteListener::Delegate {
  public:
   // The browser view's class name.
   static const char kViewClassName[];
   // Initial child indices for well-known views.
   static const int kTabstripIndex;
   static const int kInfoBarIndex;
   static const int kToolbarIndex;
 
   explicit BrowserView(Browser* browser);
   virtual ~BrowserView();
@@ skipping 342 matching lines @@
   // Returns a new LayoutManager for this browser view. A subclass may
   // override to implement different layout policy.
   virtual views::LayoutManager* CreateLayoutManager() const;
 
   // Browser window related initializations.
   virtual void Init();
 
   // Callback for the loading animation(s) associated with this view.
   virtual void LoadingAnimationCallback();
 
+  // LoadCompleteListener::Delegate implementation. Creates and initializes the
+  // |jumplist_| after the first page load.
+  virtual void OnLoadCompleted() OVERRIDE;
+
  private:
   friend class BrowserViewLayout;
   FRIEND_TEST_ALL_PREFIXES(BrowserViewsAccessibilityTest,
                            TestAboutChromeViewAccObj);
 
   enum FullscreenType {
     FOR_DESKTOP,
     FOR_METRO
   };
 
@@ skipping 223 matching lines @@
 
   // This object is initialized with the frame window HWND. This
   // object is also passed as a tick handler with the ticker_ object.
   // It is used to periodically monitor for hung plugin windows
   HungWindowDetector hung_window_detector_;
 
   // This object is invoked by hung_window_detector_ when it detects a hung
   // plugin window.
   HungPluginAction hung_plugin_action_;
 
+  // Helper class to listen for completion of first page load.
+  scoped_ptr<LoadCompleteListener> load_complete_listener_;
+
   // The custom JumpList for Windows 7.
   scoped_refptr<JumpList> jumplist_;
 #endif
 
 #if defined(USE_ASH)
   // Needs to be BrowserLauncerItemController for
   // "BrowserActivationStateChanged" and "favicon_loader".
   // TODO(skuhne): Remove once per-app is default.
   scoped_ptr<BrowserLauncherItemController> launcher_item_controller_;
 #endif
@@ skipping 17 matching lines @@
   gfx::ScopedSysColorChangeListener color_change_listener_;
 
   scoped_ptr<InstantPreviewControllerViews> preview_controller_;
 
   mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_