Changeset 67982 in webkit


Ignore:
Timestamp:
Sep 21, 2010 1:22:15 PM (14 years ago)
Author:
commit-queue@webkit.org
Message:

2010-09-21 Chris Guillory <​chris.guillory@google.com>

Reviewed by Chris Fleizach.

Send webkit accessibility notifications to Chromium.
​https://bugs.webkit.org/show_bug.cgi?id=45156

  • platform/chromium/accessibility/post-notification-ActiveDescendantChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-CheckedStateChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-ChildrenChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-FocusedUIElementChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-FocusedUIElementChanged.html:
  • platform/chromium/accessibility/post-notification-LayoutComplete-expected.txt:
  • platform/chromium/accessibility/post-notification-LayoutComplete.html:
  • platform/chromium/accessibility/post-notification-LiveRegionChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-LoadComplete-expected.txt:
  • platform/chromium/accessibility/post-notification-LoadComplete.html:
  • platform/chromium/accessibility/post-notification-MenuListValueChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-RowCollapsed-expected.txt:
  • platform/chromium/accessibility/post-notification-RowCountChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-RowExpanded-expected.txt:
  • platform/chromium/accessibility/post-notification-ScrolledToAnchor-expected.txt:
  • platform/chromium/accessibility/post-notification-ScrolledToAnchor.html:
  • platform/chromium/accessibility/post-notification-SelectedChildrenChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-SelectedTextChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-SelectedTextChanged.html:
  • platform/chromium/accessibility/post-notification-ValueChanged-expected.txt:
  • platform/chromium/accessibility/post-notification.js: (test):

2010-09-21 Chris Guillory <​chris.guillory@google.com>

Reviewed by Chris Fleizach.

Send webkit accessibility notifications to Chromium.
​https://bugs.webkit.org/show_bug.cgi?id=45156

  • WebCore.gypi:
  • accessibility/chromium/AXObjectCacheChromium.cpp: (WebCore::AXObjectCache::postPlatformNotification): (WebCore::AXObjectCache::handleFocusedUIElementChanged): (WebCore::AXObjectCache::handleScrolledToAnchor):
  • dom/Document.cpp: (WebCore::Document::implicitClose): (WebCore::Document::setFocusedNode):
  • editing/SelectionController.h:
  • editing/chromium/SelectionControllerChromium.cpp: Added. (WebCore::SelectionController::notifyAccessibilityForSelectionChange):
  • page/FrameView.cpp: (WebCore::FrameView::layout):
  • page/chromium/ChromeClientChromium.h:

2010-09-21 Chris Guillory <​chris.guillory@google.com>

Reviewed by Chris Fleizach.

Send webkit accessibility notifications to Chromium.
​https://bugs.webkit.org/show_bug.cgi?id=45156

  • public/WebAccessibilityCache.h:
  • public/WebViewClient.h:
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::focusedNodeChanged): (WebKit::ChromeClientImpl::getPopupMenuInfo):
  • src/ChromeClientImpl.h:
  • src/WebAccessibilityCache.cpp: (WebKit::WebAccessibilityCache::accessibilityEnabled):
Location:
trunk
Files:
35 edited
1 copied

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r67980 r67982  
     12010-09-21  Chris Guillory   <chris.guillory@google.com>
     2
     3        Reviewed by Chris Fleizach.
     4
     5        Send webkit accessibility notifications to Chromium.
     6        https://bugs.webkit.org/show_bug.cgi?id=45156
     7
     8        * platform/chromium/accessibility/post-notification-ActiveDescendantChanged-expected.txt:
     9        * platform/chromium/accessibility/post-notification-CheckedStateChanged-expected.txt:
     10        * platform/chromium/accessibility/post-notification-ChildrenChanged-expected.txt:
     11        * platform/chromium/accessibility/post-notification-FocusedUIElementChanged-expected.txt:
     12        * platform/chromium/accessibility/post-notification-FocusedUIElementChanged.html:
     13        * platform/chromium/accessibility/post-notification-LayoutComplete-expected.txt:
     14        * platform/chromium/accessibility/post-notification-LayoutComplete.html:
     15        * platform/chromium/accessibility/post-notification-LiveRegionChanged-expected.txt:
     16        * platform/chromium/accessibility/post-notification-LoadComplete-expected.txt:
     17        * platform/chromium/accessibility/post-notification-LoadComplete.html:
     18        * platform/chromium/accessibility/post-notification-MenuListValueChanged-expected.txt:
     19        * platform/chromium/accessibility/post-notification-RowCollapsed-expected.txt:
     20        * platform/chromium/accessibility/post-notification-RowCountChanged-expected.txt:
     21        * platform/chromium/accessibility/post-notification-RowExpanded-expected.txt:
     22        * platform/chromium/accessibility/post-notification-ScrolledToAnchor-expected.txt:
     23        * platform/chromium/accessibility/post-notification-ScrolledToAnchor.html:
     24        * platform/chromium/accessibility/post-notification-SelectedChildrenChanged-expected.txt:
     25        * platform/chromium/accessibility/post-notification-SelectedTextChanged-expected.txt:
     26        * platform/chromium/accessibility/post-notification-SelectedTextChanged.html:
     27        * platform/chromium/accessibility/post-notification-ValueChanged-expected.txt:
     28        * platform/chromium/accessibility/post-notification.js:
     29        (test):
     30
    1312010-09-21  Andrew Wilson  <atwilson@chromium.org>
    232

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-ActiveDescendantChanged-expected.txt

    r67498 r67982  
     1AccessibilityNotification - FocusedUIElementChanged - id:ul-for-ActiveDescendantChanged
     2AccessibilityNotification - FocusedUIElementChanged - id:r1
    13AccessibilityNotification - ActiveDescendantChanged - id:ul-for-ActiveDescendantChanged
     4AccessibilityNotification - LoadComplete
    25This tests that chromium correctly recieves the ActiveDescendantChanged notification.
    36li1

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-CheckedStateChanged-expected.txt

    r67498 r67982  
    11AccessibilityNotification - CheckedStateChanged - id:checkbox-for-CheckedStateChanged
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the CheckedStateChanged notification. 

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-ChildrenChanged-expected.txt

    r67498 r67982  
    11AccessibilityNotification - ChildrenChanged - id:div-for-ChildrenChanged
     2AccessibilityNotification - LayoutComplete
     3AccessibilityNotification - LoadComplete
    24This tests that chromium correctly recieves the ChildrenChanged notification.
    35text

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-FocusedUIElementChanged-expected.txt

    r67498 r67982  
     1AccessibilityNotification - FocusedUIElementChanged - id:div-for-FocusedUIElementChanged
     2AccessibilityNotification - LoadComplete
    13This tests that chromium correctly recieves the FocusedUIElementChanged notification.
     4div

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-FocusedUIElementChanged.html

    r67498 r67982  
    33<script>
    44    function testNotification() {
    5         // Not sent to the chromium port - see handleFocusedUIElementChanged
     5        document.body.children[0].focus();
    66    }
    77</script>
    … …  
    1010This tests that chromium correctly recieves the FocusedUIElementChanged notification.
    1111
     12<div id="div-for-FocusedUIElementChanged" aria-help="div-for-FocusedUIElementChanged" tabindex="-1">div</div>
     13
    1214<script type="text/javascript" src="post-notification.js"></script>
    1315</body>

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-LayoutComplete-expected.txt

    r67498 r67982  
     1AccessibilityNotification - LayoutComplete
     2AccessibilityNotification - LoadComplete
    13This tests that chromium correctly recieves the LayoutComplete notification.

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-LayoutComplete.html

    r67498 r67982  
    33<script>
    44    function testNotification() {
    5         // Mac specific
     5        // Triger layout operation.
     6        document.body.children[0].style.width = 500;
    67    }
    78</script>
    … …  
    1011This tests that chromium correctly recieves the LayoutComplete notification.
    1112
     13<div id="div-for-LayoutComplete" aria-help="div-for-LayoutComplete"><div></div></div>
     14
    1215<script type="text/javascript" src="post-notification.js"></script>
    1316</body>

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-LiveRegionChanged-expected.txt

    r67498 r67982  
    11AccessibilityNotification - LiveRegionChanged - id:ul-for-LiveRegionChanged
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the LiveRegionChanged notification.

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-LoadComplete-expected.txt

    r67498 r67982  
     1AccessibilityNotification - LoadComplete
    12This tests that chromium correctly recieves the LoadComplete notification.

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-LoadComplete.html

    r67498 r67982  
    22<head>
    33<script>
    4     function testNotification() {
    5         // Mac specific
    6     }
     4    function testNotification() { }
    75</script>
    86</head>

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-MenuListValueChanged-expected.txt

    r67498 r67982  
    11AccessibilityNotification - MenuListValueChanged - id:select-for-MenuListValueChanged
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the MenuListValueChanged notification. 

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-RowCollapsed-expected.txt

    r67498 r67982  
    11AccessibilityNotification - RowCollapsed - id:div-for-RowCollapsed
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the RowCollapsed notification.

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-RowCountChanged-expected.txt

    r67498 r67982  
    11AccessibilityNotification - RowCountChanged - id:div-for-RowCountChanged
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the RowCountChanged notification.

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-RowExpanded-expected.txt

    r67498 r67982  
    11AccessibilityNotification - RowExpanded - id:div-for-RowExpanded
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the RowExpanded notification.

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-ScrolledToAnchor-expected.txt

    r67498 r67982  
     1AccessibilityNotification - ScrolledToAnchor - id:a-for-ScrolledToAnchor
     2AccessibilityNotification - ChildrenChanged
     3AccessibilityNotification - ChildrenChanged
     4AccessibilityNotification - LayoutComplete
     5AccessibilityNotification - LoadComplete
    16This tests that chromium correctly recieves the ScrolledToAnchor notification.

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-ScrolledToAnchor.html

    r67498 r67982  
    33<script>
    44    function testNotification() {
    5         // Not sent to the chromium port - see handleScrolledToAnchor
     5        window.location.href = '#a-for-ScrolledToAnchor';
     6
     7        // Remove element to prevent another scroll notification. Appears to repro when test is run in isolation.
     8        document.body.removeChild(document.body.children[0]);
    69    }
    710</script>
    … …  
    1013This tests that chromium correctly recieves the ScrolledToAnchor notification.
    1114
     15<a id="a-for-ScrolledToAnchor" aria-help="a-for-ScrolledToAnchor"></a>
     16
    1217<script type="text/javascript" src="post-notification.js"></script>
    1318</body>

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-SelectedChildrenChanged-expected.txt

    r67498 r67982  
    11AccessibilityNotification - SelectedChildrenChanged - id:select-for-SelectedChildrenChanged
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the SelectedChildrenChanged notification. 

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-SelectedTextChanged-expected.txt

    r67498 r67982  
    1 This tests that chromium correctly recieves the SelectedTextChanged notification.
     1AccessibilityNotification - FocusedUIElementChanged - id:text-for-SelectedTextChanged
     2AccessibilityNotification - SelectedTextChanged - id:text-for-SelectedTextChanged
     3AccessibilityNotification - LoadComplete
     4This tests that chromium correctly recieves the SelectedTextChanged notification. 

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-SelectedTextChanged.html

    r67498 r67982  
    33<script>
    44    function testNotification() {
    5         // Mac specific - see notifyAccessibilityForSelectionChange
     5        document.getElementById("text-for-SelectedTextChanged").select();
    66    }
    77</script>
    … …  
    1010This tests that chromium correctly recieves the SelectedTextChanged notification.
    1111
     12<input id="text-for-SelectedTextChanged" aria-help="text-for-SelectedTextChanged" value="text" type=text />
     13
    1214<script type="text/javascript" src="post-notification.js"></script>
    1315</body>

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification-ValueChanged-expected.txt

    r67498 r67982  
    11AccessibilityNotification - ValueChanged - id:text-for-ValueChanged
     2AccessibilityNotification - LoadComplete
    23This tests that chromium correctly recieves the ValueChanged notification. 

  • trunk/LayoutTests/platform/chromium/accessibility/post-notification.js

    r67498 r67982  
    2222
    2323  // Use setTimeout so that asynchronous accessibility notifications can be processed.
    24   setTimeout(testDone, 0);
     24  setTimeout("setTimeout(testDone, 0);", 0);
    2525}

  • trunk/WebCore/ChangeLog

    r67981 r67982  
     12010-09-21  Chris Guillory   <chris.guillory@google.com>
     2
     3        Reviewed by Chris Fleizach.
     4
     5        Send webkit accessibility notifications to Chromium.
     6        https://bugs.webkit.org/show_bug.cgi?id=45156
     7
     8        * WebCore.gypi:
     9        * accessibility/chromium/AXObjectCacheChromium.cpp:
     10        (WebCore::AXObjectCache::postPlatformNotification):
     11        (WebCore::AXObjectCache::handleFocusedUIElementChanged):
     12        (WebCore::AXObjectCache::handleScrolledToAnchor):
     13        * dom/Document.cpp:
     14        (WebCore::Document::implicitClose):
     15        (WebCore::Document::setFocusedNode):
     16        * editing/SelectionController.h:
     17        * editing/chromium/SelectionControllerChromium.cpp: Added.
     18        (WebCore::SelectionController::notifyAccessibilityForSelectionChange):
     19        * page/FrameView.cpp:
     20        (WebCore::FrameView::layout):
     21        * page/chromium/ChromeClientChromium.h:
     22
    1232010-09-21  Alejandro G. Castro  <alex@igalia.com>
    224

  • trunk/WebCore/WebCore.gypi

    r67922 r67982  
    13341334            'editing/android/EditorAndroid.cpp',
    13351335            'editing/chromium/EditorChromium.cpp',
     1336            'editing/chromium/SelectionControllerChromium.cpp',
    13361337            'editing/mac/EditorMac.mm',
    13371338            'editing/mac/SelectionControllerMac.mm',

  • trunk/WebCore/accessibility/chromium/AXObjectCacheChromium.cpp

    r67498 r67982  
    6464        return;
    6565
    66     // TODO: Remove after the new postAccessibilityNotification is used downstream.
    6766    switch (notification) {
     67    case AXActiveDescendantChanged:
     68        if (!obj->document()->focusedNode() || (obj->node() != obj->document()->focusedNode()))
     69            break;
     70
     71        // Calling handleFocusedUIElementChanged will focus the new active
     72        // descendant and send the AXFocusedUIElementChanged notification.
     73        handleFocusedUIElementChanged(0, obj->document()->focusedNode()->renderer());
     74        break;
    6875    case AXCheckedStateChanged:
    69         client->didChangeAccessibilityObjectState(obj);
    70         break;
    7176    case AXChildrenChanged:
    72         client->didChangeAccessibilityObjectChildren(obj);
    73         break;
    74     case AXActiveDescendantChanged:
    7577    case AXFocusedUIElementChanged:
    7678    case AXLayoutComplete:
    … …  
    9193}
    9294
    93 void AXObjectCache::handleFocusedUIElementChanged(RenderObject*, RenderObject*)
     95void AXObjectCache::handleFocusedUIElementChanged(RenderObject*, RenderObject* newFocusedRenderer)
    9496{
     97    if (!newFocusedRenderer)
     98        return;
     99
     100    Page* page = newFocusedRenderer->document()->page();
     101    if (!page)
     102        return;
     103
     104    AccessibilityObject* focusedObject = focusedUIElementForPage(page);
     105    if (!focusedObject)
     106        return;
     107
     108    postPlatformNotification(focusedObject, AXFocusedUIElementChanged);
    95109}
    96110
    97 void AXObjectCache::handleScrolledToAnchor(const Node*)
     111void AXObjectCache::handleScrolledToAnchor(const Node* anchorNode)
    98112{
     113    // The anchor node may not be accessible. Post the notification for the
     114    // first accessible object.
     115    postPlatformNotification(AccessibilityObject::firstAccessibleObjectFromNode(anchorNode), AXScrolledToAnchor);
    99116}
    100117

  • trunk/WebCore/dom/Document.cpp

    r67898 r67982  
    20522052    }
    20532053
    2054 #if PLATFORM(MAC)
     2054#if PLATFORM(MAC) || PLATFORM(CHROMIUM)
    20552055    if (f && renderObject && this == topDocument() && AXObjectCache::accessibilityEnabled()) {
    20562056        // The AX cache may have been cleared at this point, but we need to make sure it contains an
    … …  
    31663166    }
    31673167
    3168 #if ((PLATFORM(MAC) || PLATFORM(WIN)) && !PLATFORM(CHROMIUM)) || PLATFORM(GTK)
     3168#if PLATFORM(MAC) || PLATFORM(WIN) || PLATFORM(GTK) || PLATFORM(CHROMIUM)
    31693169    if (!focusChangeBlocked && m_focusedNode && AXObjectCache::accessibilityEnabled()) {
    31703170        RenderObject* oldFocusedRenderer = 0;

  • trunk/WebCore/editing/SelectionController.h

    r67909 r67982  
    248248}
    249249
    250 #if !(PLATFORM(MAC) || PLATFORM(GTK))
     250#if !(PLATFORM(MAC) || PLATFORM(GTK) || PLATFORM(CHROMIUM))
    251251inline void SelectionController::notifyAccessibilityForSelectionChange()
    252252{

  • trunk/WebCore/editing/chromium/SelectionControllerChromium.cpp

    r67981 r67982  
    11/*
    2  * Copyright (C) 2009 Google Inc. All rights reserved.
    3  *
     2 * Copyright (C) 2010 Google Inc. All rights reserved.
     3 * 
    44 * Redistribution and use in source and binary forms, with or without
    55 * modification, are permitted provided that the following conditions are
    66 * met:
    7  *
     7 * 
    88 *     * Redistributions of source code must retain the above copyright
    99 * notice, this list of conditions and the following disclaimer.
    … …  
    1515 * contributors may be used to endorse or promote products derived from
    1616 * this software without specific prior written permission.
    17  *
     17 * 
    1818 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
    1919 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
    … …  
    3030
    3131#include "config.h"
    32 #include "WebAccessibilityCache.h"
     32#include "SelectionController.h"
    3333
    3434#include "AXObjectCache.h"
     35#include "Frame.h"
    3536
    36 using namespace WebCore;
     37namespace WebCore {
    3738
    38 namespace WebKit {
    39 
    40 void WebAccessibilityCache::enableAccessibility()
     39void SelectionController::notifyAccessibilityForSelectionChange()
    4140{
    42     AXObjectCache::enableAccessibility();
     41    // FIXME: Support editable text in chromium.
     42    if (AXObjectCache::accessibilityEnabled() && m_selection.start().isNotNull() && m_selection.end().isNotNull()) {
     43        Document* document = m_frame->document();
     44        document->axObjectCache()->postNotification(m_selection.start().node()->renderer(), AXObjectCache::AXSelectedTextChanged, false);
     45    }
    4346}
    4447
    45 }
     48} // namespace WebCore

  • trunk/WebCore/page/FrameView.cpp

    r67853 r67982  
    800800    m_layoutCount++;
    801801
    802 #if PLATFORM(MAC)
     802#if PLATFORM(MAC) || PLATFORM(CHROMIUM)
    803803    if (AXObjectCache::accessibilityEnabled())
    804804        root->document()->axObjectCache()->postNotification(root, AXObjectCache::AXLayoutComplete, true);

  • trunk/WebCore/page/chromium/ChromeClientChromium.h

    r67498 r67982  
    5555    virtual void popupClosed(PopupContainer* popupContainer) = 0;
    5656
    57     // Notifies embedder that the state of an accessibility object has changed.
    58     virtual void didChangeAccessibilityObjectState(AccessibilityObject*) = 0;
    59    
    60     // Notified embedder that the children of an accessibility object has changed.
    61     virtual void didChangeAccessibilityObjectChildren(AccessibilityObject*) = 0;
    62    
    6357    // Notifies embedder about an accessibility notification.
    6458    virtual void postAccessibilityNotification(AccessibilityObject*, AXObjectCache::AXNotification) = 0;

  • trunk/WebKit/chromium/ChangeLog

    r67949 r67982  
     12010-09-21  Chris Guillory   <chris.guillory@google.com>
     2
     3        Reviewed by Chris Fleizach.
     4
     5        Send webkit accessibility notifications to Chromium.
     6        https://bugs.webkit.org/show_bug.cgi?id=45156
     7
     8        * public/WebAccessibilityCache.h:
     9        * public/WebViewClient.h:
     10        * src/ChromeClientImpl.cpp:
     11        (WebKit::ChromeClientImpl::focusedNodeChanged):
     12        (WebKit::ChromeClientImpl::getPopupMenuInfo):
     13        * src/ChromeClientImpl.h:
     14        * src/WebAccessibilityCache.cpp:
     15        (WebKit::WebAccessibilityCache::accessibilityEnabled):
     16
    1172010-09-21  Steve Block  <steveblock@google.com>
    218

  • trunk/WebKit/chromium/public/WebAccessibilityCache.h

    r62133 r67982  
    4646    WEBKIT_API static WebAccessibilityCache* create();
    4747    WEBKIT_API static void enableAccessibility();
     48    WEBKIT_API static bool accessibilityEnabled();
    4849
    4950    virtual void initialize(WebView* view) = 0;

  • trunk/WebKit/chromium/public/WebViewClient.h

    r67498 r67982  
    277277    virtual void focusAccessibilityObject(const WebAccessibilityObject&) { }
    278278
    279     // Notifies embedder that the state of an accessibility object has changed.
    280     virtual void didChangeAccessibilityObjectState(const WebAccessibilityObject&) { }
    281    
    282     // Notifies embedder that the children of an accessibility object has changed.
    283     virtual void didChangeAccessibilityObjectChildren(const WebAccessibilityObject&) { }
    284    
    285279    // Notifies embedder about an accessibility notification.
    286280    virtual void postAccessibilityNotification(const WebAccessibilityObject&, WebAccessibilityNotification) { }

  • trunk/WebKit/chromium/src/ChromeClientImpl.cpp

    r67498 r67982  
    258258        ASSERT_NOT_REACHED();
    259259        return;
    260     }
     260    }
     261
     262    // TODO: Remove once the FocusedUIElementChanged notification is handled downstream.
    261263    if (document && document->axObjectCache()->accessibilityEnabled()) {
    262264        // Retrieve the focused AccessibilityObject.
    … …  
    761763}
    762764
    763 void ChromeClientImpl::didChangeAccessibilityObjectState(AccessibilityObject* obj)
    764 {
    765     // Alert assistive technology about the accessibility object state change
    766     if (obj)
    767         m_webView->client()->didChangeAccessibilityObjectState(WebAccessibilityObject(obj));
    768 }
    769 
    770 void ChromeClientImpl::didChangeAccessibilityObjectChildren(WebCore::AccessibilityObject* obj)
    771 {
    772     // Alert assistive technology about the accessibility object children change
    773     if (obj)
    774         m_webView->client()->didChangeAccessibilityObjectChildren(WebAccessibilityObject(obj));
    775 }
    776 
    777765void ChromeClientImpl::postAccessibilityNotification(AccessibilityObject* obj, AXObjectCache::AXNotification notification)
    778766{

  • trunk/WebKit/chromium/src/ChromeClientImpl.h

    r67498 r67982  
    167167                             bool handleExternally);
    168168    virtual void popupClosed(WebCore::PopupContainer* popupContainer);
    169     virtual void didChangeAccessibilityObjectState(WebCore::AccessibilityObject*);
    170     virtual void didChangeAccessibilityObjectChildren(WebCore::AccessibilityObject*);
    171169    virtual void postAccessibilityNotification(WebCore::AccessibilityObject*, WebCore::AXObjectCache::AXNotification);
    172170

  • trunk/WebKit/chromium/src/WebAccessibilityCache.cpp

    r50714 r67982  
    4343}
    4444
     45bool WebAccessibilityCache::accessibilityEnabled()
     46{
     47    return AXObjectCache::accessibilityEnabled();
    4548}
     49
     50}
Note: See TracChangeset for help on using the changeset viewer.