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

Side by Side Diff: chrome/browser/views/frame/browser_view.cc

Issue 15098: Enables restoring of keyboard focus to the the toolbar button that last had f... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 12 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/command_line.h" 5 #include "base/command_line.h"
6 6
7 #include "chrome/browser/views/frame/browser_view.h" 7 #include "chrome/browser/views/frame/browser_view.h"
8 8
9 #include "base/file_version_info.h" 9 #include "base/file_version_info.h"
10 #include "base/time.h" 10 #include "base/time.h"
(...skipping 483 matching lines...) Expand 10 before | Expand all | Expand 10 after
494 BrowserView* BrowserView::GetBrowserView() const { 494 BrowserView* BrowserView::GetBrowserView() const {
495 return NULL; 495 return NULL;
496 } 496 }
497 497
498 void BrowserView::UpdateToolbar(TabContents* contents, 498 void BrowserView::UpdateToolbar(TabContents* contents,
499 bool should_restore_state) { 499 bool should_restore_state) {
500 toolbar_->Update(contents, should_restore_state); 500 toolbar_->Update(contents, should_restore_state);
501 } 501 }
502 502
503 void BrowserView::FocusToolbar() { 503 void BrowserView::FocusToolbar() {
504 // Do not restore the button that previously had accessibility focus, if
505 // focus is set by using the toolbar focus keyboard shortcut.
506 toolbar_->set_acc_focused_view(NULL);
504 toolbar_->RequestFocus(); 507 toolbar_->RequestFocus();
505 } 508 }
506 509
507 void BrowserView::DestroyBrowser() { 510 void BrowserView::DestroyBrowser() {
508 // Explicitly delete the BookmarkBarView now. That way we don't have to 511 // Explicitly delete the BookmarkBarView now. That way we don't have to
509 // worry about the BookmarkBarView potentially outliving the Browser & 512 // worry about the BookmarkBarView potentially outliving the Browser &
510 // Profile. 513 // Profile.
511 bookmark_bar_view_.reset(); 514 bookmark_bar_view_.reset();
512 browser_.reset(); 515 browser_.reset();
513 } 516 }
(...skipping 835 matching lines...) Expand 10 before | Expand all | Expand 10 after
1349 1352
1350 // static 1353 // static
1351 void BrowserView::InitClass() { 1354 void BrowserView::InitClass() {
1352 static bool initialized = false; 1355 static bool initialized = false;
1353 if (!initialized) { 1356 if (!initialized) {
1354 ResourceBundle& rb = ResourceBundle::GetSharedInstance(); 1357 ResourceBundle& rb = ResourceBundle::GetSharedInstance();
1355 default_favicon_ = *rb.GetBitmapNamed(IDR_DEFAULT_FAVICON); 1358 default_favicon_ = *rb.GetBitmapNamed(IDR_DEFAULT_FAVICON);
1356 initialized = true; 1359 initialized = true;
1357 } 1360 }
1358 } 1361 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/views/toolbar_view.h » ('j') | chrome/browser/views/toolbar_view.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698