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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/accessibility/browser_accessibility_win.cc

Issue 10379029: Revert 135593 - Add initial GTK web accessibility framework (new with valgrind fix). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 7 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/accessibility/browser_accessibility_win.h ('k') | content/browser/accessibility/browser_accessibility_win_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/accessibility/browser_accessibility_win.cc
===================================================================
--- content/browser/accessibility/browser_accessibility_win.cc (revision 135594)
+++ content/browser/accessibility/browser_accessibility_win.cc (working copy)
@@ -135,7 +135,7 @@
return E_FAIL;
*target = static_cast<IAccessible*>(
- result->ToBrowserAccessibilityWin()->NewReference());
+ result->toBrowserAccessibilityWin()->NewReference());
return S_OK;
}
@@ -176,8 +176,8 @@
return instance->NewReference();
}
-BrowserAccessibilityWin* BrowserAccessibilityWin::ToBrowserAccessibilityWin() {
- return this;
+BrowserAccessibilityWin* BrowserAccessibility::toBrowserAccessibilityWin() {
+ return static_cast<BrowserAccessibilityWin*>(this);
}
BrowserAccessibilityWin::BrowserAccessibilityWin()
@@ -238,7 +238,7 @@
child->lVal = CHILDID_SELF;
} else {
child->vt = VT_DISPATCH;
- child->pdispVal = result->ToBrowserAccessibilityWin()->NewReference();
+ child->pdispVal = result->toBrowserAccessibilityWin()->NewReference();
}
return S_OK;
}
@@ -307,7 +307,7 @@
}
end->vt = VT_DISPATCH;
- end->pdispVal = result->ToBrowserAccessibilityWin()->NewReference();
+ end->pdispVal = result->toBrowserAccessibilityWin()->NewReference();
return S_OK;
}
@@ -465,13 +465,11 @@
if (!disp_parent)
return E_INVALIDARG;
- IAccessible* parent;
- if (parent_) {
- parent = parent_->ToBrowserAccessibilityWin();
- } else {
+ IAccessible* parent = parent_->toBrowserAccessibilityWin();
+ if (parent == NULL) {
// This happens if we're the root of the tree;
// return the IAccessible for the window.
- parent = manager_->ToBrowserAccessibilityManagerWin()->
+ parent = manager_->toBrowserAccessibilityManagerWin()->
GetParentWindowIAccessible();
}
@@ -568,7 +566,7 @@
if (children_[i]->HasState(WebAccessibility::STATE_SELECTED)) {
selected->vt = VT_DISPATCH;
selected->pdispVal =
- children_[i]->ToBrowserAccessibilityWin()->NewReference();
+ children_[i]->toBrowserAccessibilityWin()->NewReference();
return S_OK;
}
}
@@ -583,7 +581,7 @@
if (children_[i]->HasState(WebAccessibility::STATE_SELECTED)) {
enum_variant->ItemAt(index)->vt = VT_DISPATCH;
enum_variant->ItemAt(index)->pdispVal =
- children_[i]->ToBrowserAccessibilityWin()->NewReference();
+ children_[i]->toBrowserAccessibilityWin()->NewReference();
++index;
}
}
@@ -1074,7 +1072,8 @@
return S_FALSE;
int cell_id = unique_cell_ids_[cell_index];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
int col_index;
if (cell &&
cell->GetIntAttribute(
@@ -1180,14 +1179,15 @@
for (int i = 0; i < columns; ++i) {
int cell_id = cell_ids_[row * columns + i];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
- if (cell && cell->role() == WebAccessibility::ROLE_ROW_HEADER) {
- if (cell->name().size() > 0) {
- *description = SysAllocString(cell->name().c_str());
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
+ if (cell && cell->role_ == WebAccessibility::ROLE_ROW_HEADER) {
+ if (cell->name_.size() > 0) {
+ *description = SysAllocString(cell->name_.c_str());
return S_OK;
}
- return cell->ToBrowserAccessibilityWin()->GetStringAttributeAsBstr(
+ return cell->GetStringAttributeAsBstr(
WebAccessibility::ATTR_DESCRIPTION, description);
}
}
@@ -1218,7 +1218,8 @@
return E_INVALIDARG;
int cell_id = cell_ids_[row * columns + column];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
int rowspan;
if (cell &&
cell->GetIntAttribute(
@@ -1254,7 +1255,8 @@
return S_FALSE;
int cell_id = unique_cell_ids_[cell_index];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
int cell_row_index;
if (cell &&
cell->GetIntAttribute(
@@ -1386,7 +1388,8 @@
return S_FALSE;
int cell_id = unique_cell_ids_[index];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
int rowspan;
int colspan;
if (cell &&
@@ -1523,8 +1526,9 @@
for (int i = 0; i < rows; ++i) {
int cell_id = table->cell_ids()[i * columns + column];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
- if (cell && cell->role() == WebAccessibility::ROLE_COLUMN_HEADER)
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
+ if (cell && cell->role_ == WebAccessibility::ROLE_COLUMN_HEADER)
(*n_column_header_cells)++;
}
@@ -1533,11 +1537,11 @@
int index = 0;
for (int i = 0; i < rows; ++i) {
int cell_id = table->cell_ids()[i * columns + column];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
- if (cell && cell->role() == WebAccessibility::ROLE_COLUMN_HEADER) {
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
+ if (cell && cell->role_ == WebAccessibility::ROLE_COLUMN_HEADER) {
(*cell_accessibles)[index] =
- static_cast<IAccessible*>(
- cell->ToBrowserAccessibilityWin()->NewReference());
+ static_cast<IAccessible*>(cell->NewReference());
++index;
}
}
@@ -1620,8 +1624,9 @@
for (int i = 0; i < columns; ++i) {
int cell_id = table->cell_ids()[row * columns + i];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
- if (cell && cell->role() == WebAccessibility::ROLE_ROW_HEADER)
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
+ if (cell && cell->role_ == WebAccessibility::ROLE_ROW_HEADER)
(*n_row_header_cells)++;
}
@@ -1630,11 +1635,11 @@
int index = 0;
for (int i = 0; i < columns; ++i) {
int cell_id = table->cell_ids()[row * columns + i];
- BrowserAccessibility* cell = manager_->GetFromRendererID(cell_id);
- if (cell && cell->role() == WebAccessibility::ROLE_ROW_HEADER) {
+ BrowserAccessibilityWin* cell =
+ manager_->GetFromRendererID(cell_id)->toBrowserAccessibilityWin();
+ if (cell && cell->role_ == WebAccessibility::ROLE_ROW_HEADER) {
(*cell_accessibles)[index] =
- static_cast<IAccessible*>(
- cell->ToBrowserAccessibilityWin()->NewReference());
+ static_cast<IAccessible*>(cell->NewReference());
++index;
}
}
@@ -1732,7 +1737,7 @@
}
*table = static_cast<IAccessibleTable*>(
- find_table->ToBrowserAccessibilityWin()->NewReference());
+ find_table->toBrowserAccessibilityWin()->NewReference());
return S_OK;
}
@@ -2090,7 +2095,7 @@
}
BrowserAccessibilityWin* child =
- children_[hyperlinks_[index]]->ToBrowserAccessibilityWin();
+ children_[hyperlinks_[index]]->toBrowserAccessibilityWin();
*hyperlink = static_cast<IAccessibleHyperlink*>(child->NewReference());
return S_OK;
}
@@ -2392,12 +2397,7 @@
if (!node)
return E_INVALIDARG;
- if (!parent_) {
- *node = NULL;
- return S_FALSE;
- }
-
- *node = parent_->ToBrowserAccessibilityWin()->NewReference();
+ *node = parent_->toBrowserAccessibilityWin()->NewReference();
return S_OK;
}
@@ -2409,7 +2409,7 @@
return E_INVALIDARG;
if (children_.size()) {
- *node = children_[0]->ToBrowserAccessibilityWin()->NewReference();
+ *node = children_[0]->toBrowserAccessibilityWin()->NewReference();
return S_OK;
} else {
*node = NULL;
@@ -2425,7 +2425,7 @@
return E_INVALIDARG;
if (children_.size()) {
- *node = children_[children_.size() - 1]->ToBrowserAccessibilityWin()->
+ *node = children_[children_.size() - 1]->toBrowserAccessibilityWin()->
NewReference();
return S_OK;
} else {
@@ -2444,7 +2444,7 @@
if (parent_ && index_in_parent_ > 0) {
*node = parent_->children()[index_in_parent_ - 1]->
- ToBrowserAccessibilityWin()->NewReference();
+ toBrowserAccessibilityWin()->NewReference();
return S_OK;
} else {
*node = NULL;
@@ -2463,7 +2463,7 @@
index_in_parent_ >= 0 &&
index_in_parent_ < static_cast<int>(parent_->children().size()) - 1) {
*node = parent_->children()[index_in_parent_ + 1]->
- ToBrowserAccessibilityWin()->NewReference();
+ toBrowserAccessibilityWin()->NewReference();
return S_OK;
} else {
*node = NULL;
@@ -2481,7 +2481,7 @@
return E_INVALIDARG;
if (child_index < children_.size()) {
- *node = children_[child_index]->ToBrowserAccessibilityWin()->NewReference();
+ *node = children_[child_index]->toBrowserAccessibilityWin()->NewReference();
return S_OK;
} else {
*node = NULL;
@@ -2848,9 +2848,9 @@
return this;
if (child_id >= 1 && child_id <= static_cast<LONG>(children_.size()))
- return children_[child_id - 1]->ToBrowserAccessibilityWin();
+ return children_[child_id - 1]->toBrowserAccessibilityWin();
- return manager_->GetFromChildID(child_id)->ToBrowserAccessibilityWin();
+ return manager_->GetFromChildID(child_id)->toBrowserAccessibilityWin();
}
HRESULT BrowserAccessibilityWin::GetStringAttributeAsBstr(
@@ -2944,8 +2944,7 @@
BrowserAccessibilityWin* BrowserAccessibilityWin::GetFromRendererID(
int32 renderer_id) {
- BrowserAccessibility* result = manager_->GetFromRendererID(renderer_id);
- return result ? result->ToBrowserAccessibilityWin() : NULL;
+ return manager_->GetFromRendererID(renderer_id)->toBrowserAccessibilityWin();
}
void BrowserAccessibilityWin::InitRoleAndState() {
« no previous file with comments | « content/browser/accessibility/browser_accessibility_win.h ('k') | content/browser/accessibility/browser_accessibility_win_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698