Implement table and table cell
Created attachment 445554 [details] Patch
Comment on attachment 445554 [details] Patch Other than changing a few pointer literals to “nulptr” before landing, patch LGTM. View in context: https://bugs.webkit.org/attachment.cgi?id=445554&action=review > Tools/WebKitTestRunner/InjectedBundle/atspi/AccessibilityUIElementAtspi.cpp:1069 > + return JSStringCreateWithCharacters(0, 0); While changing this you could replace the first zero with nullptr: return JSStringCreateWithCharacters(nullptr, 0); =] > Tools/WebKitTestRunner/InjectedBundle/atspi/AccessibilityUIElementAtspi.cpp:1078 > + return JSStringCreateWithCharacters(0, 0); Same here: s/0/nullptr > Tools/WebKitTestRunner/InjectedBundle/atspi/AccessibilityUIElementAtspi.cpp:1092 > + return JSStringCreateWithCharacters(0, 0); ...and here... > Tools/WebKitTestRunner/InjectedBundle/atspi/AccessibilityUIElementAtspi.cpp:1101 > + return JSStringCreateWithCharacters(0, 0); ...one more... > Tools/WebKitTestRunner/InjectedBundle/atspi/AccessibilityUIElementAtspi.cpp:1146 > + return JSStringCreateWithCharacters(0, 0); ...and another... (okay, I will stop commenting now on each of those)
Committed r286416 (244762@main): <https://commits.webkit.org/244762@main>