Patch to follow.
Created attachment 100968 [details] Patch
Comment on attachment 100968 [details] Patch Attachment 100968 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/9094260 New failing tests: inspector/elements/event-listener-sidebar.html inspector/debugger/error-in-watch-expressions.html
Created attachment 100971 [details] Archive of layout-test-results from ec2-cr-linux-03 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-03 Port: Chromium Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment on attachment 100968 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=100968&action=review Let's make sure that it works for object property section as well, r- for now. > Source/WebCore/inspector/front-end/ObjectPropertiesSection.js:81 > + var expandableArrayThreashold = 100; typo: threashold -> threshold > Source/WebCore/inspector/front-end/ObjectPropertiesSection.js:90 > + if (index >= 0 && (index < minIndex || minIndex == -1)) if -> else if? == -> ===
Created attachment 101725 [details] Patch
Created attachment 101729 [details] [IMAGE] Looks with patch applied.
Comment on attachment 101725 [details] Patch Array UI is not properly updated after editing array value. r- for this.
*** Bug 80021 has been marked as a duplicate of this bug. ***
Created attachment 129694 [details] Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Attachment 129694 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'LayoutTests/ChangeLog', u'La..." exit_code: 1 WARNING: File exempt from style guide. Skipping: "Source/WebKit2/UIProcess/API/gtk/WebKitDefines.h" Source/WebKit2/WebProcess/gtk/WebProcessMainGtk.cpp:71: Use 0 instead of NULL. [readability/null] [5] Total errors found: 1 in 186 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 129694 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=129694&action=review Looks like this needs a test. > Source/WebCore/inspector/front-end/ConsoleMessage.js:291 > + if (array.arrayLength() > 100) Maybe use const? > Source/WebCore/inspector/front-end/ObjectPropertiesSection.js:382 > + TreeElement.call(this, "[" + this._properties[fromIndex].name + " \u2026 " + this._properties[toIndex - 1].name + "]", null, true); var title = fromIndex === toIndex - 1 ? "[" + this._properties[fromIndex].name + " \u2026 " + this._properties[toIndex - 1].name + "]" : this._properties[fromIndex].name; TreeElement.call(this, title, null, true); > Source/WebCore/inspector/front-end/ObjectPropertiesSection.js:404 > + treeElement._readOnly = true; So we can not edit any array shown as dir(array) even if it shorter than 20 elements? > Source/WebCore/inspector/front-end/RemoteObject.js:313 > + return parseInt(this._description.match(/[0-9]+/), 10); Maybe var matches = this._description.match(/\[([0-9]+)\]/)) return parseInt(matches[1], 10); would be better?
Created attachment 129713 [details] Patch
This was landed as r109372