Status: Untriaged
Owner: [email protected]
CC: [email protected]
Labels: Type-Cleanup Pri-3 OS-All Area-BrowserUI Size-Small

New issue 16231 by [email protected]: Browser::GetCurrentPageTitle  
has a misleading name and comment
http://code.google.com/p/chromium/issues/detail?id=16231

(just using the bug tracker as a little TODO list)

   // Gets the title of the page in the selected tab.
   std::wstring GetCurrentPageTitle() const;

However, it is in fact the *window* title. I was recently writing a browser
test involving that function, and got confused for a few minutes.

Maybe GetCurrentWindowTitle or just GetWindowTitle would be a better name?

If you don't like the "rename the function" idea, I think that at least the
comment should be changed.

What do you think?

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to