Adds MSAA information to various (previously uncovered) bits of the UI, inclu...

Adds MSAA information to various (previously uncovered) bits of the UI, including Bookmark and Extensions infobars. This is to moving towards completing the hierarchy for the benefit of full screen reader support.

BUG=None.
Test=None.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=21476

[jcampan, 2009-07-14 16:44:24]

LGTM with nits

http://codereview.chromium.org/155446/diff/1021/35
File chrome/browser/views/bookmark_bar_view.cc (right):

http://codereview.chromium.org/155446/diff/1021/35#newcode344
Line 344: virtual bool GetAccessibleName(std::wstring* name) {
Nit: DCHECK(name)

http://codereview.chromium.org/155446/diff/1021/35#newcode346
Line 346: (*name).assign(accessible_name_);
Nit: name-> instead of (*name).

http://codereview.chromium.org/155446/diff/1021/35#newcode833
Line 833: (*name).assign(accessible_name_);
Same nits as above: CHECK(name) and name->

http://codereview.chromium.org/155446/diff/1021/28
File chrome/browser/views/extensions/extension_shelf.cc (right):

http://codereview.chromium.org/155446/diff/1021/28#newcode576
Line 576: if (!accessible_name_.empty()) {
Same nit as previously: DCHECK(name), name->assign

http://codereview.chromium.org/155446/diff/1021/26
File chrome/browser/views/infobars/infobar_container.cc (right):

http://codereview.chromium.org/155446/diff/1021/26#newcode76
Line 76: if (!accessible_name_.empty()) {
Same nit as previously mentioned: DCHECK(name), name->assign

http://codereview.chromium.org/155446/diff/1021/27
File chrome/browser/views/infobars/infobar_container.h (right):

http://codereview.chromium.org/155446/diff/1021/27#newcode42
Line 42: protected:
Nit: Add a blank line above "protected:"

http://codereview.chromium.org/155446/diff/1021/27#newcode75
Line 75: // Storage of strings needed for accessibility.
Nit: strings -> the string

http://codereview.chromium.org/155446/diff/1021/30
File chrome/browser/views/tab_contents/tab_contents_container.cc (right):

http://codereview.chromium.org/155446/diff/1021/30#newcode86
Line 86: if (!accessible_name_.empty()) {
Same nit as previously mentioned.