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

Side by Side Diff: third_party/WebKit/Source/modules/accessibility/AXTable.cpp

Issue 2657443005: Migrate WTF::HashSet::add() to ::insert() [part 1 of N] (Closed)
Patch Set: Created 3 years, 10 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2008 Apple Inc. All rights reserved. 2 * Copyright (C) 2008 Apple Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 401 matching lines...) Expand 10 before | Expand all | Expand 10 after
412 AXTableRow* row = toAXTableRow(rowObject); 412 AXTableRow* row = toAXTableRow(rowObject);
413 // We need to check every cell for a new row, because cell spans 413 // We need to check every cell for a new row, because cell spans
414 // can cause us to miss rows if we just check the first column. 414 // can cause us to miss rows if we just check the first column.
415 if (appendedRows.contains(row)) 415 if (appendedRows.contains(row))
416 continue; 416 continue;
417 417
418 row->setRowIndex(static_cast<int>(m_rows.size())); 418 row->setRowIndex(static_cast<int>(m_rows.size()));
419 m_rows.push_back(row); 419 m_rows.push_back(row);
420 if (!row->accessibilityIsIgnored()) 420 if (!row->accessibilityIsIgnored())
421 m_children.push_back(row); 421 m_children.push_back(row);
422 appendedRows.add(row); 422 appendedRows.insert(row);
423 } 423 }
424 424
425 tableSection = table->sectionBelow(tableSection, SkipEmptySections); 425 tableSection = table->sectionBelow(tableSection, SkipEmptySections);
426 } 426 }
427 427
428 // make the columns based on the number of columns in the first body 428 // make the columns based on the number of columns in the first body
429 unsigned length = initialTableSection->numEffectiveColumns(); 429 unsigned length = initialTableSection->numEffectiveColumns();
430 for (unsigned i = 0; i < length; ++i) { 430 for (unsigned i = 0; i < length; ++i) {
431 AXTableColumn* column = toAXTableColumn(axCache.getOrCreate(ColumnRole)); 431 AXTableColumn* column = toAXTableColumn(axCache.getOrCreate(ColumnRole));
432 column->setColumnIndex((int)i); 432 column->setColumnIndex((int)i);
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after
604 } 604 }
605 605
606 DEFINE_TRACE(AXTable) { 606 DEFINE_TRACE(AXTable) {
607 visitor->trace(m_rows); 607 visitor->trace(m_rows);
608 visitor->trace(m_columns); 608 visitor->trace(m_columns);
609 visitor->trace(m_headerContainer); 609 visitor->trace(m_headerContainer);
610 AXLayoutObject::trace(visitor); 610 AXLayoutObject::trace(visitor);
611 } 611 }
612 612
613 } // namespace blink 613 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698