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

Side by Side Diff: third_party/WebKit/Source/web/WebAXObject.cpp

Issue 1839643009: RELEASE_ASSERT -> CHECK and ASSERT -> DCHECK in web. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix compilation. Created 4 years, 8 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) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google 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 are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * 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 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 614 matching lines...) Expand 10 before | Expand all | Expand 10 after
625 // from Chromium. http://crbug.com/489590 625 // from Chromium. http://crbug.com/489590
626 626
627 return false; 627 return false;
628 } 628 }
629 629
630 WebRect WebAXObject::boundingBoxRect() const 630 WebRect WebAXObject::boundingBoxRect() const
631 { 631 {
632 if (isDetached()) 632 if (isDetached())
633 return WebRect(); 633 return WebRect();
634 634
635 ASSERT(isLayoutClean(m_private->getDocument())); 635 #if ENABLE(ASSERT)
tkent 2016/03/31 23:05:21 Please don't add ENABLE(ASSERT). isLayoutClean() s
636 DCHECK(isLayoutClean(m_private->getDocument()));
637 #endif
636 638
637 return pixelSnappedIntRect(m_private->elementRect()); 639 return pixelSnappedIntRect(m_private->elementRect());
638 } 640 }
639 641
640 WebString WebAXObject::fontFamily() const 642 WebString WebAXObject::fontFamily() const
641 { 643 {
642 if (isDetached()) 644 if (isDetached())
643 return WebString(); 645 return WebString();
644 646
645 return m_private->fontFamily(); 647 return m_private->fontFamily();
(...skipping 787 matching lines...) Expand 10 before | Expand all | Expand 10 after
1433 { 1435 {
1434 if (isDetached()) 1436 if (isDetached())
1435 return; 1437 return;
1436 1438
1437 Vector<AXObject::AXRange> wordBoundaries; 1439 Vector<AXObject::AXRange> wordBoundaries;
1438 m_private->wordBoundaries(wordBoundaries); 1440 m_private->wordBoundaries(wordBoundaries);
1439 1441
1440 WebVector<int> wordStartOffsets(wordBoundaries.size()); 1442 WebVector<int> wordStartOffsets(wordBoundaries.size());
1441 WebVector<int> wordEndOffsets(wordBoundaries.size()); 1443 WebVector<int> wordEndOffsets(wordBoundaries.size());
1442 for (size_t i = 0; i < wordBoundaries.size(); ++i) { 1444 for (size_t i = 0; i < wordBoundaries.size(); ++i) {
1443 ASSERT(wordBoundaries[i].isSimple()); 1445 DCHECK(wordBoundaries[i].isSimple());
1444 wordStartOffsets[i] = wordBoundaries[i].anchorOffset; 1446 wordStartOffsets[i] = wordBoundaries[i].anchorOffset;
1445 wordEndOffsets[i] = wordBoundaries[i].focusOffset; 1447 wordEndOffsets[i] = wordBoundaries[i].focusOffset;
1446 } 1448 }
1447 1449
1448 starts.swap(wordStartOffsets); 1450 starts.swap(wordStartOffsets);
1449 ends.swap(wordEndOffsets); 1451 ends.swap(wordEndOffsets);
1450 } 1452 }
1451 1453
1452 bool WebAXObject::isScrollableContainer() const 1454 bool WebAXObject::isScrollableContainer() const
1453 { 1455 {
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
1525 m_private = object; 1527 m_private = object;
1526 return *this; 1528 return *this;
1527 } 1529 }
1528 1530
1529 WebAXObject::operator AXObject*() const 1531 WebAXObject::operator AXObject*() const
1530 { 1532 {
1531 return m_private.get(); 1533 return m_private.get();
1532 } 1534 }
1533 1535
1534 } // namespace blink 1536 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698