Chromium Code Reviews| Index: chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc |
| =================================================================== |
| --- chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc (revision 91698) |
| +++ chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc (working copy) |
| @@ -195,6 +195,15 @@ |
| } |
| } |
| + bool GetTooltipText(const gfx::Point& p, std::wstring* tooltip) { |
|
Alexei Svitkine (slow)
2011/07/07 19:58:56
Please use virtual and OVERRIDE here (like |IsTrig
PhilBeaudoin
2011/07/07 20:26:17
Done.
|
| + gfx::Point location(p); |
|
sky
2011/07/07 20:07:16
It doesn't look 199 and 200 are needed.
PhilBeaudoin
2011/07/07 20:26:17
Done.
|
| + ConvertPointToScreen(this, &location); |
| + if (text_size_.width() > GetTextBounds().width()) { |
|
sky
2011/07/07 20:07:16
no {}
PhilBeaudoin
2011/07/07 20:26:17
Done.
|
| + *tooltip = text_; |
| + } |
| + return !tooltip->empty(); |
| + } |
| + |
| virtual bool IsTriggerableEvent(const views::MouseEvent& e) OVERRIDE { |
| // Left clicks should show the menu contents and right clicks should show |
| // the context menu. They should not trigger the opening of underlying urls. |