OLD | NEW |
---|---|
1 // Copyright (c) 2010 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/views/toolbar_view.h" | 5 #include "chrome/browser/ui/views/toolbar_view.h" |
6 | 6 |
7 #include "app/l10n_util.h" | 7 #include "app/l10n_util.h" |
8 #include "app/resource_bundle.h" | 8 #include "app/resource_bundle.h" |
9 #include "base/i18n/number_formatting.h" | 9 #include "base/i18n/number_formatting.h" |
10 #include "chrome/app/chrome_command_ids.h" | 10 #include "chrome/app/chrome_command_ids.h" |
11 #include "chrome/browser/accessibility/browser_accessibility_state.h" | 11 #include "chrome/browser/accessibility/browser_accessibility_state.h" |
12 #include "chrome/browser/background_page_tracker.h" | 12 #include "chrome/browser/background_page_tracker.h" |
13 #include "chrome/browser/prefs/pref_service.h" | 13 #include "chrome/browser/prefs/pref_service.h" |
14 #include "chrome/browser/profiles/profile.h" | 14 #include "chrome/browser/profiles/profile.h" |
15 #include "chrome/browser/themes/browser_theme_provider.h" | 15 #include "chrome/browser/themes/browser_theme_provider.h" |
(...skipping 19 matching lines...) Expand all Loading... | |
35 #include "views/widget/tooltip_manager.h" | 35 #include "views/widget/tooltip_manager.h" |
36 #include "views/window/non_client_view.h" | 36 #include "views/window/non_client_view.h" |
37 #include "views/window/window.h" | 37 #include "views/window/window.h" |
38 | 38 |
39 #if defined(OS_CHROMEOS) | 39 #if defined(OS_CHROMEOS) |
40 #include "chrome/browser/chromeos/cros/cros_library.h" | 40 #include "chrome/browser/chromeos/cros/cros_library.h" |
41 #include "chrome/browser/chromeos/cros/update_library.h" | 41 #include "chrome/browser/chromeos/cros/update_library.h" |
42 #include "chrome/browser/chromeos/dom_ui/wrench_menu_ui.h" | 42 #include "chrome/browser/chromeos/dom_ui/wrench_menu_ui.h" |
43 #include "views/controls/menu/menu_2.h" | 43 #include "views/controls/menu/menu_2.h" |
44 #endif | 44 #endif |
45 #include "chrome/browser/views/wrench_menu.h" | 45 #include "chrome/browser/ui/views/wrench_menu.h" |
46 | 46 |
47 #if defined(OS_WIN) | 47 #if defined(OS_WIN) |
48 #include "chrome/browser/enumerate_modules_model_win.h" | 48 #include "chrome/browser/enumerate_modules_model_win.h" |
49 #endif | 49 #endif |
50 | 50 |
51 // The space between items is 4 px in general. | 51 // The space between items is 4 px in general. |
52 const int ToolbarView::kStandardSpacing = 4; | 52 const int ToolbarView::kStandardSpacing = 4; |
53 // The top of the toolbar has an edge we have to skip over in addition to the 4 | 53 // The top of the toolbar has an edge we have to skip over in addition to the 4 |
54 // px of spacing. | 54 // px of spacing. |
55 const int ToolbarView::kVertSpacing = kStandardSpacing + 1; | 55 const int ToolbarView::kVertSpacing = kStandardSpacing + 1; |
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
341 #endif | 341 #endif |
342 wrench_menu_ = new WrenchMenu(browser_); | 342 wrench_menu_ = new WrenchMenu(browser_); |
343 wrench_menu_->Init(wrench_menu_model_.get()); | 343 wrench_menu_->Init(wrench_menu_model_.get()); |
344 | 344 |
345 for (size_t i = 0; i < menu_listeners_.size(); ++i) | 345 for (size_t i = 0; i < menu_listeners_.size(); ++i) |
346 menu_listeners_[i]->OnMenuOpened(); | 346 menu_listeners_[i]->OnMenuOpened(); |
347 | 347 |
348 wrench_menu_->RunMenu(app_menu_); | 348 wrench_menu_->RunMenu(app_menu_); |
349 | 349 |
350 #if defined(OS_CHROMEOS) | 350 #if defined(OS_CHROMEOS) |
351 cleanup: | 351 cleanup: |
msw
2011/01/10 00:38:29
I got this pre-submit error:
/cygdrive/d/src/chrom
Ben Goodger (Google)
2011/01/10 17:09:30
Simple, just indent 1 space before cleanup:. e.g.
msw
2011/01/10 17:54:01
Done. The codebase has both, 1 and 0 spaces...
Ben Goodger (Google)
2011/01/10 17:55:02
According to the error then, the latter is wrong :
| |
352 #endif | 352 #endif |
353 if (destroyed_flag) | 353 if (destroyed_flag) |
354 return; | 354 return; |
355 destroyed_flag_ = NULL; | 355 destroyed_flag_ = NULL; |
356 | 356 |
357 // Stop showing the background app badge also. | 357 // Stop showing the background app badge also. |
358 BackgroundPageTracker::GetInstance()->AcknowledgeBackgroundPages(); | 358 BackgroundPageTracker::GetInstance()->AcknowledgeBackgroundPages(); |
359 } | 359 } |
360 | 360 |
361 //////////////////////////////////////////////////////////////////////////////// | 361 //////////////////////////////////////////////////////////////////////////////// |
(...skipping 400 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
762 SkBitmap* badge = tp->GetBitmapNamed(IDR_BACKGROUND_BADGE); | 762 SkBitmap* badge = tp->GetBitmapNamed(IDR_BACKGROUND_BADGE); |
763 string16 badge_text = base::FormatNumber( | 763 string16 badge_text = base::FormatNumber( |
764 BackgroundPageTracker::GetInstance()->GetBackgroundPageCount()); | 764 BackgroundPageTracker::GetInstance()->GetBackgroundPageCount()); |
765 return badge_util::DrawBadgeIconOverlay( | 765 return badge_util::DrawBadgeIconOverlay( |
766 *badge, | 766 *badge, |
767 kBadgeTextFontSize, | 767 kBadgeTextFontSize, |
768 badge_text, | 768 badge_text, |
769 l10n_util::GetStringUTF16(IDS_BACKGROUND_PAGE_BADGE_OVERFLOW)); | 769 l10n_util::GetStringUTF16(IDS_BACKGROUND_PAGE_BADGE_OVERFLOW)); |
770 } | 770 } |
771 | 771 |
OLD | NEW |