⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 126932 in webkit


Ignore:
Timestamp:
Aug 28, 2012, 2:58:35 PM (14 years ago)
Author:
dmazzoni@google.com
Message:

AX: Crash due to object getting deleted inside updateBackingStore
​https://bugs.webkit.org/show_bug.cgi?id=94619

Reviewed by Chris Fleizach.

Source/WebKit/chromium:

Chromium now calls updateBackingStoreAndCheckValidity explicitly,
so we can now get rid of calls to updateBackingStore in the
method implementations, and just make sure we're checking isDetached.

  • public/WebAccessibilityObject.h:

(WebAccessibilityObject):

  • src/WebAccessibilityObject.cpp:

(WebKit::WebAccessibilityObject::axID):
(WebKit::WebAccessibilityObject::accessibilityDescription):
(WebKit::WebAccessibilityObject::actionVerb):
(WebKit::WebAccessibilityObject::canSetFocusAttribute):
(WebKit::WebAccessibilityObject::canSetValueAttribute):
(WebKit::WebAccessibilityObject::childCount):
(WebKit::WebAccessibilityObject::childAt):
(WebKit::WebAccessibilityObject::firstChild):
(WebKit::WebAccessibilityObject::focusedChild):
(WebKit::WebAccessibilityObject::lastChild):
(WebKit::WebAccessibilityObject::nextSibling):
(WebKit::WebAccessibilityObject::parentObject):
(WebKit::WebAccessibilityObject::previousSibling):
(WebKit::WebAccessibilityObject::canSetSelectedAttribute):
(WebKit::WebAccessibilityObject::isAnchor):
(WebKit::WebAccessibilityObject::isAriaReadOnly):
(WebKit::WebAccessibilityObject::isButtonStateMixed):
(WebKit::WebAccessibilityObject::isChecked):
(WebKit::WebAccessibilityObject::isCollapsed):
(WebKit::WebAccessibilityObject::isControl):
(WebKit::WebAccessibilityObject::isEnabled):
(WebKit::WebAccessibilityObject::isFocused):
(WebKit::WebAccessibilityObject::isHovered):
(WebKit::WebAccessibilityObject::isIndeterminate):
(WebKit::WebAccessibilityObject::isLinked):
(WebKit::WebAccessibilityObject::isLoaded):
(WebKit::WebAccessibilityObject::isMultiSelectable):
(WebKit::WebAccessibilityObject::isOffScreen):
(WebKit::WebAccessibilityObject::isPasswordField):
(WebKit::WebAccessibilityObject::isPressed):
(WebKit::WebAccessibilityObject::isReadOnly):
(WebKit::WebAccessibilityObject::isRequired):
(WebKit::WebAccessibilityObject::isSelected):
(WebKit::WebAccessibilityObject::isSelectedOptionActive):
(WebKit::WebAccessibilityObject::isVertical):
(WebKit::WebAccessibilityObject::isVisible):
(WebKit::WebAccessibilityObject::isVisited):
(WebKit::WebAccessibilityObject::accessKey):
(WebKit::WebAccessibilityObject::ariaHasPopup):
(WebKit::WebAccessibilityObject::ariaLiveRegionAtomic):
(WebKit::WebAccessibilityObject::ariaLiveRegionBusy):
(WebKit::WebAccessibilityObject::ariaLiveRegionRelevant):
(WebKit::WebAccessibilityObject::ariaLiveRegionStatus):
(WebKit::WebAccessibilityObject::boundingBoxRect):
(WebKit::WebAccessibilityObject::estimatedLoadingProgress):
(WebKit::WebAccessibilityObject::helpText):
(WebKit::WebAccessibilityObject::headingLevel):
(WebKit::WebAccessibilityObject::hierarchicalLevel):
(WebKit::WebAccessibilityObject::hitTest):
(WebKit::WebAccessibilityObject::keyboardShortcut):
(WebKit::WebAccessibilityObject::performDefaultAction):
(WebKit::WebAccessibilityObject::roleValue):
(WebKit::WebAccessibilityObject::selectionEnd):
(WebKit::WebAccessibilityObject::selectionStart):
(WebKit::WebAccessibilityObject::stringValue):
(WebKit::WebAccessibilityObject::title):
(WebKit::WebAccessibilityObject::titleUIElement):
(WebKit::WebAccessibilityObject::url):
(WebKit::WebAccessibilityObject::valueDescription):
(WebKit::WebAccessibilityObject::valueForRange):
(WebKit::WebAccessibilityObject::maxValueForRange):
(WebKit::WebAccessibilityObject::minValueForRange):
(WebKit::WebAccessibilityObject::node):
(WebKit::WebAccessibilityObject::document):
(WebKit::WebAccessibilityObject::accessibilityIsIgnored):
(WebKit::WebAccessibilityObject::lineBreaks):
(WebKit::WebAccessibilityObject::columnCount):
(WebKit::WebAccessibilityObject::rowCount):
(WebKit::WebAccessibilityObject::cellForColumnAndRow):
(WebKit::WebAccessibilityObject::cellColumnIndex):
(WebKit::WebAccessibilityObject::cellColumnSpan):
(WebKit::WebAccessibilityObject::cellRowIndex):
(WebKit::WebAccessibilityObject::cellRowSpan):
(WebKit::WebAccessibilityObject::scrollToMakeVisible):
(WebKit::WebAccessibilityObject::scrollToMakeVisibleWithSubFocus):
(WebKit::WebAccessibilityObject::scrollToGlobalPoint):

Tools:

Change isValid to !isDetached after deleting isValid from
chromium WebAccessibilityObject as being redundant.

  • DumpRenderTree/chromium/TestRunner/AccessibilityUIElementChromium.cpp:

(AccessibilityUIElement::isValidGetterCallback):

Location:
trunk
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebKit/chromium/ChangeLog

    r126926 r126932  
     12012-08-28  Dominic Mazzoni  <dmazzoni@google.com>
     2
     3        AX: Crash due to object getting deleted inside updateBackingStore
     4        https://bugs.webkit.org/show_bug.cgi?id=94619
     5
     6        Reviewed by Chris Fleizach.
     7
     8        Chromium now calls updateBackingStoreAndCheckValidity explicitly,
     9        so we can now get rid of calls to updateBackingStore in the
     10        method implementations, and just make sure we're checking isDetached.
     11
     12        * public/WebAccessibilityObject.h:
     13        (WebAccessibilityObject):
     14        * src/WebAccessibilityObject.cpp:
     15        (WebKit::WebAccessibilityObject::axID):
     16        (WebKit::WebAccessibilityObject::accessibilityDescription):
     17        (WebKit::WebAccessibilityObject::actionVerb):
     18        (WebKit::WebAccessibilityObject::canSetFocusAttribute):
     19        (WebKit::WebAccessibilityObject::canSetValueAttribute):
     20        (WebKit::WebAccessibilityObject::childCount):
     21        (WebKit::WebAccessibilityObject::childAt):
     22        (WebKit::WebAccessibilityObject::firstChild):
     23        (WebKit::WebAccessibilityObject::focusedChild):
     24        (WebKit::WebAccessibilityObject::lastChild):
     25        (WebKit::WebAccessibilityObject::nextSibling):
     26        (WebKit::WebAccessibilityObject::parentObject):
     27        (WebKit::WebAccessibilityObject::previousSibling):
     28        (WebKit::WebAccessibilityObject::canSetSelectedAttribute):
     29        (WebKit::WebAccessibilityObject::isAnchor):
     30        (WebKit::WebAccessibilityObject::isAriaReadOnly):
     31        (WebKit::WebAccessibilityObject::isButtonStateMixed):
     32        (WebKit::WebAccessibilityObject::isChecked):
     33        (WebKit::WebAccessibilityObject::isCollapsed):
     34        (WebKit::WebAccessibilityObject::isControl):
     35        (WebKit::WebAccessibilityObject::isEnabled):
     36        (WebKit::WebAccessibilityObject::isFocused):
     37        (WebKit::WebAccessibilityObject::isHovered):
     38        (WebKit::WebAccessibilityObject::isIndeterminate):
     39        (WebKit::WebAccessibilityObject::isLinked):
     40        (WebKit::WebAccessibilityObject::isLoaded):
     41        (WebKit::WebAccessibilityObject::isMultiSelectable):
     42        (WebKit::WebAccessibilityObject::isOffScreen):
     43        (WebKit::WebAccessibilityObject::isPasswordField):
     44        (WebKit::WebAccessibilityObject::isPressed):
     45        (WebKit::WebAccessibilityObject::isReadOnly):
     46        (WebKit::WebAccessibilityObject::isRequired):
     47        (WebKit::WebAccessibilityObject::isSelected):
     48        (WebKit::WebAccessibilityObject::isSelectedOptionActive):
     49        (WebKit::WebAccessibilityObject::isVertical):
     50        (WebKit::WebAccessibilityObject::isVisible):
     51        (WebKit::WebAccessibilityObject::isVisited):
     52        (WebKit::WebAccessibilityObject::accessKey):
     53        (WebKit::WebAccessibilityObject::ariaHasPopup):
     54        (WebKit::WebAccessibilityObject::ariaLiveRegionAtomic):
     55        (WebKit::WebAccessibilityObject::ariaLiveRegionBusy):
     56        (WebKit::WebAccessibilityObject::ariaLiveRegionRelevant):
     57        (WebKit::WebAccessibilityObject::ariaLiveRegionStatus):
     58        (WebKit::WebAccessibilityObject::boundingBoxRect):
     59        (WebKit::WebAccessibilityObject::estimatedLoadingProgress):
     60        (WebKit::WebAccessibilityObject::helpText):
     61        (WebKit::WebAccessibilityObject::headingLevel):
     62        (WebKit::WebAccessibilityObject::hierarchicalLevel):
     63        (WebKit::WebAccessibilityObject::hitTest):
     64        (WebKit::WebAccessibilityObject::keyboardShortcut):
     65        (WebKit::WebAccessibilityObject::performDefaultAction):
     66        (WebKit::WebAccessibilityObject::roleValue):
     67        (WebKit::WebAccessibilityObject::selectionEnd):
     68        (WebKit::WebAccessibilityObject::selectionStart):
     69        (WebKit::WebAccessibilityObject::stringValue):
     70        (WebKit::WebAccessibilityObject::title):
     71        (WebKit::WebAccessibilityObject::titleUIElement):
     72        (WebKit::WebAccessibilityObject::url):
     73        (WebKit::WebAccessibilityObject::valueDescription):
     74        (WebKit::WebAccessibilityObject::valueForRange):
     75        (WebKit::WebAccessibilityObject::maxValueForRange):
     76        (WebKit::WebAccessibilityObject::minValueForRange):
     77        (WebKit::WebAccessibilityObject::node):
     78        (WebKit::WebAccessibilityObject::document):
     79        (WebKit::WebAccessibilityObject::accessibilityIsIgnored):
     80        (WebKit::WebAccessibilityObject::lineBreaks):
     81        (WebKit::WebAccessibilityObject::columnCount):
     82        (WebKit::WebAccessibilityObject::rowCount):
     83        (WebKit::WebAccessibilityObject::cellForColumnAndRow):
     84        (WebKit::WebAccessibilityObject::cellColumnIndex):
     85        (WebKit::WebAccessibilityObject::cellColumnSpan):
     86        (WebKit::WebAccessibilityObject::cellRowIndex):
     87        (WebKit::WebAccessibilityObject::cellRowSpan):
     88        (WebKit::WebAccessibilityObject::scrollToMakeVisible):
     89        (WebKit::WebAccessibilityObject::scrollToMakeVisibleWithSubFocus):
     90        (WebKit::WebAccessibilityObject::scrollToGlobalPoint):
     91
    1922012-08-28  Sheriff Bot  <webkit.review.bot@gmail.com>
    293

  • trunk/Source/WebKit/chromium/public/WebAccessibilityObject.h

    r126662 r126932  
    9090    WEBKIT_EXPORT bool canSetFocusAttribute() const;
    9191    WEBKIT_EXPORT bool canSetValueAttribute() const;
    92     WEBKIT_EXPORT bool isValid() const;
    9392
    9493    WEBKIT_EXPORT unsigned childCount() const;

  • trunk/Source/WebKit/chromium/src/WebAccessibilityObject.cpp

    r126662 r126932  
    9696        return -1;
    9797
    98     m_private->updateBackingStore();
    9998    return m_private->axObjectID();
    10099}
    … …  
    112111        return WebString();
    113112
    114     m_private->updateBackingStore();
    115113    return m_private->accessibilityDescription();
    116114}
    … …  
    121119        return WebString();
    122120
    123     m_private->updateBackingStore();
    124121    return m_private->actionVerb();
    125122}
    … …  
    130127        return false;
    131128
    132     m_private->updateBackingStore();
    133129    return m_private->canSetFocusAttribute();
    134130}
    … …  
    139135        return false;
    140136
    141     m_private->updateBackingStore();
    142137    return m_private->canSetValueAttribute();
    143138}
    144139
    145 bool WebAccessibilityObject::isValid() const
    146 {
    147     if (isDetached())
    148         return false;
    149 
    150     m_private->updateBackingStore();
    151     return m_private->axObjectID();
    152 }
    153 
    154140unsigned WebAccessibilityObject::childCount() const
    155141{
    … …  
    157143        return 0;
    158144
    159     m_private->updateBackingStore();
    160145    return m_private->children().size();
    161146}
    … …  
    166151        return WebAccessibilityObject();
    167152
    168     m_private->updateBackingStore();
    169153    if (m_private->children().size() <= index)
    170154        return WebAccessibilityObject();
    … …  
    178162        return WebAccessibilityObject();
    179163
    180     m_private->updateBackingStore();
    181164    return WebAccessibilityObject(m_private->firstChild());
    182165}
    … …  
    187170        return WebAccessibilityObject();
    188171
    189     m_private->updateBackingStore();
    190172    RefPtr<AccessibilityObject> focused = m_private->focusedUIElement();
    191173    if (m_private.get() == focused.get() || m_private.get() == focused->parentObject())
    … …  
    200182        return WebAccessibilityObject();
    201183
    202     m_private->updateBackingStore();
    203184    return WebAccessibilityObject(m_private->lastChild());
    204185}
    … …  
    210191        return WebAccessibilityObject();
    211192
    212     m_private->updateBackingStore();
    213193    return WebAccessibilityObject(m_private->nextSibling());
    214194}
    … …  
    219199        return WebAccessibilityObject();
    220200
    221     m_private->updateBackingStore();
    222201    return WebAccessibilityObject(m_private->parentObject());
    223202}
    … …  
    229208        return WebAccessibilityObject();
    230209
    231     m_private->updateBackingStore();
    232210    return WebAccessibilityObject(m_private->previousSibling());
    233211}
    … …  
    238216        return 0;
    239217
    240     m_private->updateBackingStore();
    241218    return m_private->canSetSelectedAttribute();
    242219}
    … …  
    247224        return 0;
    248225
    249     m_private->updateBackingStore();
    250226    return m_private->isAnchor();
    251227}
    … …  
    256232        return 0;
    257233
    258     m_private->updateBackingStore();
    259234    return equalIgnoringCase(m_private->getAttribute(HTMLNames::aria_readonlyAttr), "true");
    260235}
    … …  
    265240        return 0;
    266241
    267     m_private->updateBackingStore();
    268242    return m_private->checkboxOrRadioValue() == ButtonStateMixed;
    269243}
    … …  
    274248        return 0;
    275249
    276     m_private->updateBackingStore();
    277250    return m_private->isChecked();
    278251}
    … …  
    283256        return 0;
    284257
    285     m_private->updateBackingStore();
    286258    return m_private->isCollapsed();
    287259}
    … …  
    292264        return 0;
    293265
    294     m_private->updateBackingStore();
    295266    return m_private->isControl();
    296267}
    … …  
    301272        return 0;
    302273
    303     m_private->updateBackingStore();
    304274    return m_private->isEnabled();
    305275}
    … …  
    310280        return 0;
    311281
    312     m_private->updateBackingStore();
    313282    return m_private->isFocused();
    314283}
    … …  
    319288        return 0;
    320289
    321     m_private->updateBackingStore();
    322290    return m_private->isHovered();
    323291}
    … …  
    328296        return 0;
    329297
    330     m_private->updateBackingStore();
    331298    return m_private->isIndeterminate();
    332299}
    … …  
    337304        return 0;
    338305
    339     m_private->updateBackingStore();
    340306    return m_private->isLinked();
    341307}
    … …  
    346312        return 0;
    347313
    348     m_private->updateBackingStore();
    349314    return m_private->isLoaded();
    350315}
    … …  
    355320        return 0;
    356321
    357     m_private->updateBackingStore();
    358322    return m_private->isMultiSelectable();
    359323}
    … …  
    364328        return 0;
    365329
    366     m_private->updateBackingStore();
    367330    return m_private->isOffScreen();
    368331}
    … …  
    373336        return 0;
    374337
    375     m_private->updateBackingStore();
    376338    return m_private->isPasswordField();
    377339}
    … …  
    382344        return 0;
    383345
    384     m_private->updateBackingStore();
    385346    return m_private->isPressed();
    386347}
    … …  
    391352        return 0;
    392353
    393     m_private->updateBackingStore();
    394354    return m_private->isReadOnly();
    395355}
    … …  
    400360        return 0;
    401361
    402     m_private->updateBackingStore();
    403362    return m_private->isRequired();
    404363}
    … …  
    409368        return 0;
    410369
    411     m_private->updateBackingStore();
    412370    return m_private->isSelected();
    413371}
    … …  
    418376        return false;
    419377
    420     m_private->updateBackingStore();
    421378    return m_private->isSelectedOptionActive();
    422379}
    … …  
    427384        return 0;
    428385
    429     m_private->updateBackingStore();
    430386    return m_private->orientation() == AccessibilityOrientationVertical;
    431387}
    … …  
    436392        return 0;
    437393
    438     m_private->updateBackingStore();
    439394    return m_private->isVisible();
    440395}
    … …  
    445400        return 0;
    446401
    447     m_private->updateBackingStore();
    448402    return m_private->isVisited();
    449403}
    … …  
    454408        return WebString();
    455409
    456     m_private->updateBackingStore();
    457410    return WebString(m_private->accessKey());
    458411}
    … …  
    463416        return 0;
    464417
    465     m_private->updateBackingStore();
    466418    return m_private->ariaHasPopup();
    467419}
    … …  
    472424        return 0;
    473425
    474     m_private->updateBackingStore();
    475426    return m_private->ariaLiveRegionAtomic();
    476427}
    … …  
    481432        return 0;
    482433
    483     m_private->updateBackingStore();
    484434    return m_private->ariaLiveRegionBusy();
    485435}
    … …  
    490440        return WebString();
    491441
    492     m_private->updateBackingStore();
    493442    return m_private->ariaLiveRegionRelevant();
    494443}
    … …  
    499448        return WebString();
    500449
    501     m_private->updateBackingStore();
    502450    return m_private->ariaLiveRegionStatus();
    503451}
    … …  
    508456        return WebRect();
    509457
    510     m_private->updateBackingStore();
    511458    return m_private->pixelSnappedBoundingBoxRect();
    512459}
    … …  
    517464        return 0.0;
    518465
    519     m_private->updateBackingStore();
    520466    return m_private->estimatedLoadingProgress();
    521467}
    … …  
    526472        return WebString();
    527473
    528     m_private->updateBackingStore();
    529474    return m_private->helpText();
    530475}
    … …  
    535480        return 0;
    536481
    537     m_private->updateBackingStore();
    538482    return m_private->headingLevel();
    539483}
    … …  
    544488        return 0;
    545489
    546     m_private->updateBackingStore();
    547490    return m_private->hierarchicalLevel();
    548491}
    … …  
    553496        return WebAccessibilityObject();
    554497
    555     m_private->updateBackingStore();
    556498    IntPoint contentsPoint = m_private->documentFrameView()->windowToContents(point);
    557499    RefPtr<AccessibilityObject> hit = m_private->accessibilityHitTest(contentsPoint);
    … …  
    571513        return WebString();
    572514
    573     m_private->updateBackingStore();
    574515    String accessKey = m_private->accessKey();
    575516    if (accessKey.isNull())
    … …  
    602543    UserGestureIndicator gestureIndicator(DefinitelyProcessingUserGesture);
    603544
    604     m_private->updateBackingStore();
    605545    return m_private->performDefaultAction();
    606546}
    … …  
    611551        return WebKit::WebAccessibilityRoleUnknown;
    612552
    613     m_private->updateBackingStore();
    614553    return static_cast<WebAccessibilityRole>(m_private->roleValue());
    615554}
    … …  
    620559        return 0;
    621560
    622     m_private->updateBackingStore();
    623561    return m_private->selectedTextRange().start + m_private->selectedTextRange().length;
    624562}
    … …  
    629567        return 0;
    630568
    631     m_private->updateBackingStore();
    632569    return m_private->selectedTextRange().start;
    633570}
    … …  
    644581        return WebString();
    645582
    646     m_private->updateBackingStore();
    647583    return m_private->stringValue();
    648584}
    … …  
    653589        return WebString();
    654590
    655     m_private->updateBackingStore();
    656591    return m_private->title();
    657592}
    … …  
    662597        return WebAccessibilityObject();
    663598
    664     m_private->updateBackingStore();
    665599    return WebAccessibilityObject(m_private->titleUIElement());
    666600}
    … …  
    671605        return WebURL();
    672606   
    673     m_private->updateBackingStore();
    674607    return m_private->url();
    675608}
    … …  
    680613        return WebString();
    681614
    682     m_private->updateBackingStore();
    683615    return m_private->valueDescription();
    684616}
    … …  
    689621        return 0.0;
    690622
    691     m_private->updateBackingStore();
    692623    return m_private->valueForRange();
    693624}
    … …  
    698629        return 0.0;
    699630
    700     m_private->updateBackingStore();
    701631    return m_private->maxValueForRange();
    702632}
    … …  
    707637        return 0.0;
    708638
    709     m_private->updateBackingStore();
    710639    return m_private->minValueForRange();
    711640}
    … …  
    715644    if (isDetached())
    716645        return WebNode();
    717 
    718     m_private->updateBackingStore();
    719646
    720647    Node* node = m_private->node();
    … …  
    730657        return WebDocument();
    731658
    732     m_private->updateBackingStore();
    733 
    734659    Document* document = m_private->document();
    735660    if (!document)
    … …  
    780705        return false;
    781706
    782     m_private->updateBackingStore();
    783707    return m_private->accessibilityIsIgnored();
    784708}
    … …  
    789713        return false;
    790714
    791     m_private->updateBackingStore();
    792715    int textLength = m_private->textLength();
    793716    if (!textLength)
    … …  
    813736        return false;
    814737
    815     m_private->updateBackingStore();
    816738    if (!m_private->isAccessibilityTable())
    817739        return 0;
    … …  
    825747        return false;
    826748
    827     m_private->updateBackingStore();
    828749    if (!m_private->isAccessibilityTable())
    829750        return 0;
    … …  
    834755WebAccessibilityObject WebAccessibilityObject::cellForColumnAndRow(unsigned column, unsigned row) const
    835756{
    836     m_private->updateBackingStore();
     757    if (isDetached())
     758        return WebAccessibilityObject();
     759
    837760    if (!m_private->isAccessibilityTable())
    838761        return WebAccessibilityObject();
    … …  
    844767unsigned WebAccessibilityObject::cellColumnIndex() const
    845768{
    846     m_private->updateBackingStore();
     769    if (isDetached())
     770        return 0;
     771
    847772    if (!m_private->isTableCell())
    848773       return 0;
    … …  
    855780unsigned WebAccessibilityObject::cellColumnSpan() const
    856781{
    857     m_private->updateBackingStore();
     782    if (isDetached())
     783        return 0;
     784
    858785    if (!m_private->isTableCell())
    859786       return 0;
    … …  
    866793unsigned WebAccessibilityObject::cellRowIndex() const
    867794{
    868     m_private->updateBackingStore();
     795    if (isDetached())
     796        return 0;
     797
    869798    if (!m_private->isTableCell())
    870799       return 0;
    … …  
    877806unsigned WebAccessibilityObject::cellRowSpan() const
    878807{
    879     m_private->updateBackingStore();
     808    if (isDetached())
     809        return 0;
     810
    880811    if (!m_private->isTableCell())
    881812       return 0;
    … …  
    888819void WebAccessibilityObject::scrollToMakeVisible() const
    889820{
    890     m_private->updateBackingStore();
    891     m_private->scrollToMakeVisible();
     821    if (!isDetached())
     822        m_private->scrollToMakeVisible();
    892823}
    893824
    894825void WebAccessibilityObject::scrollToMakeVisibleWithSubFocus(const WebRect& subfocus) const
    895826{
    896     m_private->updateBackingStore();
    897     m_private->scrollToMakeVisibleWithSubFocus(subfocus);
     827    if (!isDetached())
     828        m_private->scrollToMakeVisibleWithSubFocus(subfocus);
    898829}
    899830
    900831void WebAccessibilityObject::scrollToGlobalPoint(const WebPoint& point) const
    901832{
    902     m_private->updateBackingStore();
    903     m_private->scrollToGlobalPoint(point);
     833    if (!isDetached())
     834        m_private->scrollToGlobalPoint(point);
    904835}
    905836

  • trunk/Tools/ChangeLog

    r126926 r126932  
     12012-08-28  Dominic Mazzoni  <dmazzoni@google.com>
     2
     3        AX: Crash due to object getting deleted inside updateBackingStore
     4        https://bugs.webkit.org/show_bug.cgi?id=94619
     5
     6        Reviewed by Chris Fleizach.
     7
     8        Change isValid to !isDetached after deleting isValid from
     9        chromium WebAccessibilityObject as being redundant.
     10
     11        * DumpRenderTree/chromium/TestRunner/AccessibilityUIElementChromium.cpp:
     12        (AccessibilityUIElement::isValidGetterCallback):
     13
    1142012-08-28  Sheriff Bot  <webkit.review.bot@gmail.com>
    215

  • trunk/Tools/DumpRenderTree/chromium/TestRunner/AccessibilityUIElementChromium.cpp

    r126369 r126932  
    548548void AccessibilityUIElement::isValidGetterCallback(CppVariant* result)
    549549{
    550     result->set(accessibilityObject().isValid());
     550    result->set(!accessibilityObject().isDetached());
    551551}
    552552
Note: See TracChangeset for help on using the changeset viewer.