Improve large tab strip by leveraging touch icons when present

chrome/browser/ui/touch/tabs/touch_tab_strip_controller.h

+// 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.
+
+#ifndef CHROME_BROWSER_UI_TOUCH_TABS_TOUCH_TAB_STRIP_CONTROLLER_H_
+#define CHROME_BROWSER_UI_TOUCH_TABS_TOUCH_TAB_STRIP_CONTROLLER_H_
+#pragma once
+
+#include <map>
+#include <vector>
+
+#include "chrome/browser/favicon/favicon_service.h"
+#include "chrome/browser/history/history.h"
+#include "chrome/browser/ui/views/tabs/browser_tab_strip_controller.h"
+
+// Subclassing BrowserTabStripController to provide touch specific behavior.
+class TouchTabStripController: public BrowserTabStripController {
+ public:
+  TouchTabStripController(Browser* browser, TabStripModel* model);
+  virtual ~TouchTabStripController();
+
+  virtual void TabChangedAt(TabContentsWrapper* contents,
+                            int model_index,
+                            TabChangeType change_type) OVERRIDE;
+
+ protected:
+  virtual void SetTabRendererDataFromModel(TabContents* contents,
+                                           int model_index,
+                                           TabRendererData* data) OVERRIDE;
+
+ private:
+  void OnTouchIconAvailable(FaviconService::Handle h,
+                            history::FaviconData favicon);
+
+  // A vector of all GURL requests and the model index to be refreshed
+  typedef std::pair<GURL, int> RequestInfo;

[sky, 2011/06/10 15:38:18]

typedefs should be first in a section
(http://www.corp.google.com/eng/doc/cppguide.xml#Declaration_Order)

[Emmanuel Saint-loubert-Bié, 2011/06/10 16:09:10]

sorry I did cut and paste from another similar declaration

+  std::vector<RequestInfo> gurl_vector_;
+
+  typedef std::map<GURL, SkBitmap> TouchIconMap;
+  TouchIconMap touch_icon_map_;
+
+  // Our consumer for touch icon requests (takes gurl_vector_ index as data)
+  CancelableRequestConsumerT<int, 0> consumer_;
+
+  DISALLOW_COPY_AND_ASSIGN(TouchTabStripController);
+};
+
+#endif  // CHROME_BROWSER_UI_TOUCH_TABS_TOUCH_TAB_STRIP_CONTROLLER_H_