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

Unified Diff: chrome/browser/tabs/tab_strip_model.h

Issue 99224: Reverting 14948. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/browser.cc ('k') | chrome/browser/tabs/tab_strip_model.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/tabs/tab_strip_model.h
===================================================================
--- chrome/browser/tabs/tab_strip_model.h (revision 14948)
+++ chrome/browser/tabs/tab_strip_model.h (working copy)
@@ -109,7 +109,7 @@
virtual TabContents* AddBlankTab(bool foreground) = 0;
virtual TabContents* AddBlankTabAt(int index, bool foreground) = 0;
- // Asks for a new TabStripModel to be created and the given tab contents to
+ // Ask for a new TabStripModel to be created and the given tab contents to
// be added to it. Its size and position are reflected in |window_bounds|.
// If |dock_info|'s type is other than NONE, the newly created window should
// be docked as identified by |dock_info|. Returns the Browser object
@@ -128,7 +128,7 @@
TAB_TEAROFF_ACTION = 2
};
- // Determines what drag actions are possible for the specified strip.
+ // Determine what drag actions are possible for the specified strip.
virtual int GetDragActions() const = 0;
// Creates an appropriate TabContents for the given URL. This is handled by
@@ -144,10 +144,10 @@
bool defer_load,
SiteInstance* instance) const = 0;
- // Returns whether some contents can be duplicated.
+ // Return whether some contents can be duplicated.
virtual bool CanDuplicateContentsAt(int index) = 0;
- // Duplicates the contents at the provided index and places it into its own
+ // Duplicate the contents at the provided index and places it into its own
// window.
virtual void DuplicateContentsAt(int index) = 0;
@@ -171,9 +171,6 @@
// Restores the last closed tab if CanRestoreTab would return true.
virtual void RestoreTab() = 0;
-
- // Returns whether some contents can be closed.
- virtual bool CanCloseContentsAt(int index) = 0;
};
////////////////////////////////////////////////////////////////////////////////
@@ -257,9 +254,8 @@
// Closes the TabContents at the specified index. This causes the TabContents
// to be destroyed, but it may not happen immediately (e.g. if it's a
// WebContents).
- // Returns true if the TabContents was closed immediately, false if it was not
- // closed (we may be waiting for a response from an onunload handler, or
- // waiting for the user to confirm closure).
+ // Returns true if the TabContents was closed immediately, false if we are
+ // waiting for a response from an onunload handler.
bool CloseTabContentsAt(int index) {
return InternalCloseTabContentsAt(index, true);
}
« no previous file with comments | « chrome/browser/browser.cc ('k') | chrome/browser/tabs/tab_strip_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698