diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-05 21:22:03 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-05 21:22:03 +0000 |
| commit | 914c69f33f38bb472e5cbf8a3ab1cb3ee3a2948e (patch) | |
| tree | 2d7fa294dc42a0dcb41242ae82f34c414418a1d5 /chrome/browser/views/frame/browser_view.cc | |
| parent | 389f493751d6edde8321e18ff8b9558a72598c9e (diff) | |
| download | chromium_src-914c69f33f38bb472e5cbf8a3ab1cb3ee3a2948e.zip chromium_src-914c69f33f38bb472e5cbf8a3ab1cb3ee3a2948e.tar.gz chromium_src-914c69f33f38bb472e5cbf8a3ab1cb3ee3a2948e.tar.bz2 | |
The new history and download tab caused a regression where opening the NTP would not focus the location bar.
DOM UI now by default focus the page.
The NTP one focus the location bar.
Also removed some unused fields from the history and download DOM UI.
BUG=8356
TEST=Open NTP, focus should be on location bar
Review URL: http://codereview.chromium.org/39173
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11034 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/frame/browser_view.cc')
| -rw-r--r-- | chrome/browser/views/frame/browser_view.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/chrome/browser/views/frame/browser_view.cc b/chrome/browser/views/frame/browser_view.cc index fc0bca3..9f8ae58 100644 --- a/chrome/browser/views/frame/browser_view.cc +++ b/chrome/browser/views/frame/browser_view.cc @@ -968,7 +968,9 @@ std::wstring BrowserView::GetWindowTitle() const { } views::View* BrowserView::GetInitiallyFocusedView() { - return toolbar_->GetLocationBarView(); + // We set the frame not focus on creation so this should never be called. + NOTREACHED(); + return NULL; } bool BrowserView::ShouldShowWindowTitle() const { |