Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(208)

Side by Side Diff: chrome/browser/chromeos/wm_overview_controller.cc

Issue 6735042: Move favicon from TabContents to TabContentsWrapper. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 9 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/wm_overview_controller.h" 5 #include "chrome/browser/chromeos/wm_overview_controller.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/memory/linked_ptr.h" 10 #include "base/memory/linked_ptr.h"
11 #include "chrome/browser/browser_process.h" 11 #include "chrome/browser/browser_process.h"
12 #include "chrome/browser/chromeos/wm_ipc.h" 12 #include "chrome/browser/chromeos/wm_ipc.h"
13 #include "chrome/browser/chromeos/wm_overview_favicon.h" 13 #include "chrome/browser/chromeos/wm_overview_favicon.h"
14 #include "chrome/browser/chromeos/wm_overview_snapshot.h" 14 #include "chrome/browser/chromeos/wm_overview_snapshot.h"
15 #include "chrome/browser/chromeos/wm_overview_title.h" 15 #include "chrome/browser/chromeos/wm_overview_title.h"
16 #include "chrome/browser/favicon_tab_helper.h"
16 #include "chrome/browser/tab_contents/thumbnail_generator.h" 17 #include "chrome/browser/tab_contents/thumbnail_generator.h"
17 #include "chrome/browser/tabs/tab_strip_model.h" 18 #include "chrome/browser/tabs/tab_strip_model.h"
18 #include "chrome/browser/ui/browser.h" 19 #include "chrome/browser/ui/browser.h"
19 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h" 20 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
20 #include "chrome/browser/ui/views/frame/browser_view.h" 21 #include "chrome/browser/ui/views/frame/browser_view.h"
21 #include "content/browser/renderer_host/render_view_host.h" 22 #include "content/browser/renderer_host/render_view_host.h"
22 #include "content/browser/renderer_host/render_widget_host.h" 23 #include "content/browser/renderer_host/render_widget_host.h"
23 #include "content/browser/renderer_host/render_widget_host_view.h" 24 #include "content/browser/renderer_host/render_widget_host_view.h"
24 #include "content/browser/tab_contents/tab_contents.h" 25 #include "content/browser/tab_contents/tab_contents.h"
25 #include "content/browser/tab_contents/tab_contents_view.h" 26 #include "content/browser/tab_contents/tab_contents_view.h"
(...skipping 206 matching lines...) Expand 10 before | Expand all | Expand 10 after
232 } 233 }
233 234
234 void BrowserListener::TabChangedAt( 235 void BrowserListener::TabChangedAt(
235 TabContentsWrapper* contents, 236 TabContentsWrapper* contents,
236 int index, 237 int index,
237 TabStripModelObserver::TabChangeType change_type) { 238 TabStripModelObserver::TabChangeType change_type) {
238 if (change_type != TabStripModelObserver::LOADING_ONLY) { 239 if (change_type != TabStripModelObserver::LOADING_ONLY) {
239 snapshots_[index].title->SetTitle(contents->tab_contents()->GetTitle()); 240 snapshots_[index].title->SetTitle(contents->tab_contents()->GetTitle());
240 snapshots_[index].title->SetUrl(contents->tab_contents()->GetURL()); 241 snapshots_[index].title->SetUrl(contents->tab_contents()->GetURL());
241 snapshots_[index].favicon->SetFavicon( 242 snapshots_[index].favicon->SetFavicon(
242 contents->tab_contents()->GetFavicon()); 243 contents->favicon_tab_helper()->GetFavicon());
243 if (change_type != TabStripModelObserver::TITLE_NOT_LOADING) 244 if (change_type != TabStripModelObserver::TITLE_NOT_LOADING)
244 MarkSnapshotAsDirty(index); 245 MarkSnapshotAsDirty(index);
245 } 246 }
246 } 247 }
247 248
248 void BrowserListener::TabStripEmpty() { 249 void BrowserListener::TabStripEmpty() {
249 snapshots_.clear(); 250 snapshots_.clear();
250 } 251 }
251 252
252 void BrowserListener::TabSelectedAt(TabContentsWrapper* old_contents, 253 void BrowserListener::TabSelectedAt(TabContentsWrapper* old_contents,
(...skipping 164 matching lines...) Expand 10 before | Expand all | Expand 10 after
417 } 418 }
418 419
419 void BrowserListener::InsertSnapshot(int index) { 420 void BrowserListener::InsertSnapshot(int index) {
420 SnapshotNode node; 421 SnapshotNode node;
421 node.snapshot = new WmOverviewSnapshot; 422 node.snapshot = new WmOverviewSnapshot;
422 gfx::Size cell_size = CalculateCellSize(); 423 gfx::Size cell_size = CalculateCellSize();
423 node.snapshot->Init(cell_size, browser_, index); 424 node.snapshot->Init(cell_size, browser_, index);
424 425
425 node.favicon = new WmOverviewFavicon; 426 node.favicon = new WmOverviewFavicon;
426 node.favicon->Init(node.snapshot); 427 node.favicon->Init(node.snapshot);
427 node.favicon->SetFavicon(browser_->GetTabContentsAt(index)->GetFavicon()); 428 node.favicon->SetFavicon(
429 browser_->GetTabContentsWrapperAt(index)->
430 favicon_tab_helper()->GetFavicon());
428 431
429 node.title = new WmOverviewTitle; 432 node.title = new WmOverviewTitle;
430 node.title->Init(gfx::Size(std::max(0, cell_size.width() - 433 node.title->Init(gfx::Size(std::max(0, cell_size.width() -
431 WmOverviewFavicon::kIconSize - 434 WmOverviewFavicon::kIconSize -
432 kFaviconPadding), 435 kFaviconPadding),
433 kTitleHeight), node.snapshot); 436 kTitleHeight), node.snapshot);
434 node.title->SetTitle(browser_->GetTabContentsAt(index)->GetTitle()); 437 node.title->SetTitle(browser_->GetTabContentsAt(index)->GetTitle());
435 438
436 snapshots_.insert(snapshots_.begin() + index, node); 439 snapshots_.insert(snapshots_.begin() + index, node);
437 node.snapshot->reload_snapshot(); 440 node.snapshot->reload_snapshot();
(...skipping 300 matching lines...) Expand 10 before | Expand all | Expand 10 after
738 if (item.get() == NULL) { 741 if (item.get() == NULL) {
739 item = BrowserListenerVector::value_type( 742 item = BrowserListenerVector::value_type(
740 new BrowserListener(*iterator, this)); 743 new BrowserListener(*iterator, this));
741 } 744 }
742 listeners_.push_back(item); 745 listeners_.push_back(item);
743 ++iterator; 746 ++iterator;
744 } 747 }
745 } 748 }
746 749
747 } // namespace chromeos 750 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_edit_unittest.cc ('k') | chrome/browser/content_setting_bubble_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698