From 999aa609b3fc8a9bc14d0d39d34890cf1df1160c Mon Sep 17 00:00:00 2001
From: "ben@chromium.org"
 <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Tue, 11 Nov 2008 21:57:56 +0000
Subject: More cleanup in Browser:

- remove "RunSimpleFrameMenu" method that isn't used anymore.
- rename Browser::GetType() to Browser::type()


Review URL: http://codereview.chromium.org/10605

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5209 0039d316-1c4b-4281-b951-d872f2087c98
---
 chrome/browser/session_service.cc | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'chrome/browser/session_service.cc')

diff --git a/chrome/browser/session_service.cc b/chrome/browser/session_service.cc
index 813b56a..d1b848b 100644
--- a/chrome/browser/session_service.cc
+++ b/chrome/browser/session_service.cc
@@ -476,7 +476,7 @@ void SessionService::Observe(NotificationType type,
     case NOTIFY_BROWSER_OPENED: {
       Browser* browser = Source<Browser>(source).ptr();
       if (browser->profile() != profile_ ||
-          !should_track_changes_for_browser_type(browser->GetType())) {
+          !should_track_changes_for_browser_type(browser->type())) {
         return;
       }
 
@@ -494,7 +494,7 @@ void SessionService::Observe(NotificationType type,
               profile_, browser, false, false, false, std::vector<GURL>());
         }
       }
-      SetWindowType(browser->session_id(), browser->GetType());
+      SetWindowType(browser->session_id(), browser->type());
       break;
     }
 
@@ -1092,7 +1092,7 @@ void SessionService::BuildCommandsForBrowser(
                                    browser->window()->IsMaximized()));
 
   commands->push_back(CreateSetWindowTypeCommand(
-      browser->session_id(), browser->GetType()));
+      browser->session_id(), browser->type()));
 
   bool added_to_windows_to_track = false;
   for (int i = 0; i < browser->tab_count(); ++i) {
@@ -1119,7 +1119,7 @@ void SessionService::BuildCommandsFromBrowsers(
   DCHECK(commands);
   for (BrowserList::const_iterator i = BrowserList::begin();
        i != BrowserList::end(); ++i) {
-    if (should_track_changes_for_browser_type((*i)->GetType())) {
+    if (should_track_changes_for_browser_type((*i)->type())) {
       BuildCommandsForBrowser(*i, commands, tab_to_available_range,
                               windows_to_track);
     }
@@ -1258,7 +1258,7 @@ bool SessionService::IsOnlyOneTabLeft() {
   for (BrowserList::const_iterator i = BrowserList::begin();
        i != BrowserList::end(); ++i) {
     const SessionID::id_type window_id = (*i)->session_id().id();
-    if (should_track_changes_for_browser_type((*i)->GetType()) &&
+    if (should_track_changes_for_browser_type((*i)->type()) &&
         (*i)->profile()->GetOriginalProfile() == profile_ &&
         window_closing_ids_.find(window_id) == window_closing_ids_.end()) {
       if (++window_count > 1)
@@ -1286,7 +1286,7 @@ bool SessionService::HasOpenTabbedBrowsers(const SessionID& window_id) {
     const SessionID::id_type browser_id = browser->session_id().id();
     if (browser_id != window_id.id() &&
         window_closing_ids_.find(browser_id) == window_closing_ids_.end() &&
-        should_track_changes_for_browser_type(browser->GetType()) &&
+        should_track_changes_for_browser_type(browser->type()) &&
         browser->profile()->GetOriginalProfile() == profile_) {
       return true;
     }
-- 
cgit v1.1