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

Unified Diff: chrome/browser/ui/panels/panel_browser_view_browsertest.cc

Issue 7633034: Use Titlebar instead of TitleBar consistently in panels code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Code review feedback. Created 9 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/panels/panel_browser_view_browsertest.cc
diff --git a/chrome/browser/ui/panels/panel_browser_view_browsertest.cc b/chrome/browser/ui/panels/panel_browser_view_browsertest.cc
index de58f12091b6375e4b26866ae0554ccdddc25641..927a0825a7d7e4c545f9f751979d7f8f8be991b0 100644
--- a/chrome/browser/ui/panels/panel_browser_view_browsertest.cc
+++ b/chrome/browser/ui/panels/panel_browser_view_browsertest.cc
@@ -266,26 +266,26 @@ class PanelBrowserViewTest : public InProcessBrowserTest {
PanelManager* panel_manager = PanelManager::GetInstance();
- panel_manager->BringUpOrDownTitleBarForAllMinimizedPanels(true);
+ panel_manager->BringUpOrDownTitlebarForAllMinimizedPanels(true);
EXPECT_EQ(Panel::TITLE_ONLY, panel1->expansion_state());
EXPECT_EQ(Panel::EXPANDED, panel2->expansion_state());
EXPECT_EQ(Panel::TITLE_ONLY, panel3->expansion_state());
- panel_manager->BringUpOrDownTitleBarForAllMinimizedPanels(false);
+ panel_manager->BringUpOrDownTitlebarForAllMinimizedPanels(false);
EXPECT_EQ(Panel::MINIMIZED, panel1->expansion_state());
EXPECT_EQ(Panel::EXPANDED, panel2->expansion_state());
EXPECT_EQ(Panel::MINIMIZED, panel3->expansion_state());
// Test if it is OK to bring up title-bar given the mouse position.
- EXPECT_TRUE(panel_manager->ShouldBringUpTitleBarForAllMinimizedPanels(
+ EXPECT_TRUE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
panel1->GetBounds().x(), panel1->GetBounds().y()));
- EXPECT_FALSE(panel_manager->ShouldBringUpTitleBarForAllMinimizedPanels(
+ EXPECT_FALSE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
panel2->GetBounds().x(), panel2->GetBounds().y()));
- EXPECT_TRUE(panel_manager->ShouldBringUpTitleBarForAllMinimizedPanels(
+ EXPECT_TRUE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
panel3->GetBounds().right() - 1, panel3->GetBounds().bottom() - 1));
- EXPECT_TRUE(panel_manager->ShouldBringUpTitleBarForAllMinimizedPanels(
+ EXPECT_TRUE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
panel3->GetBounds().right() - 1, panel3->GetBounds().bottom() + 10));
- EXPECT_FALSE(panel_manager->ShouldBringUpTitleBarForAllMinimizedPanels(
+ EXPECT_FALSE(panel_manager->ShouldBringUpTitlebarForAllMinimizedPanels(
0, 0));
panel1->Close();
@@ -338,11 +338,11 @@ class PanelBrowserViewTest : public InProcessBrowserTest {
// Test that we cannot bring up other minimized panel if the mouse is over
// the panel that draws attension.
EXPECT_FALSE(PanelManager::GetInstance()->
- ShouldBringUpTitleBarForAllMinimizedPanels(
+ ShouldBringUpTitlebarForAllMinimizedPanels(
panel->GetBounds().x(), panel->GetBounds().y()));
// Test that we cannot bring down the panel that is drawing the attention.
- PanelManager::GetInstance()->BringUpOrDownTitleBarForAllMinimizedPanels(
+ PanelManager::GetInstance()->BringUpOrDownTitlebarForAllMinimizedPanels(
false);
EXPECT_EQ(Panel::TITLE_ONLY, panel->expansion_state());
@@ -468,12 +468,12 @@ IN_PROC_BROWSER_TEST_F(PanelBrowserViewTest, SetBoundsAnimation) {
// Validates that no animation should be triggered for the panel currently
// being dragged.
- browser_view->OnTitleBarMousePressed(gfx::Point(
+ browser_view->OnTitlebarMousePressed(gfx::Point(
target_bounds.x(), target_bounds.y()));
- browser_view->OnTitleBarMouseDragged(gfx::Point(
+ browser_view->OnTitlebarMouseDragged(gfx::Point(
target_bounds.x() + 5, target_bounds.y() + 5));
EXPECT_FALSE(browser_view->bounds_animator_->is_animating());
- browser_view->OnTitleBarMouseCaptureLost();
+ browser_view->OnTitlebarMouseCaptureLost();
browser_view->panel()->Close();
}
« no previous file with comments | « chrome/browser/ui/panels/panel_browser_view.cc ('k') | chrome/browser/ui/panels/panel_browser_window_cocoa.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698