[Webkit-unassigned] [Bug 44251] Web Inspector: add DOM breakpoints test

Fri Aug 20 07:06:06 PDT 2010

https://bugs.webkit.org/show_bug.cgi?id=44251

Yury Semikhatsky <yurys at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #64829|review?                     |review-
               Flag|                            |

--- Comment #2 from Yury Semikhatsky <yurys at chromium.org>  2010-08-20 07:06:07 PST ---
(From update of attachment 64829)
LayoutTests/platform/qt/Skipped:109
 +  inspector/dom-breakpoints.html
It should be skipped on Gtk too, right?

LayoutTests/http/tests/inspector/inspector-test2.js:54
 +  InspectorTest.waitUntilPaused = function(callback)
Would be nice if we could install these methods only for tests that need them like we currently do with console/elements/debugger... tests

LayoutTests/http/tests/inspector/inspector-test2.js:209
 +  function runTestInFrontend(initializeInspectorTest, completeTestCallId, testFunction)
Please move it inside runTest to not confuse people that could try to invoke this function in the inspected page.

LayoutTests/inspector/dom-breakpoints.html:20
 +          InspectorTest.addResult("Found div element.");
This test is going to fail id debugger is not always enabled, r- for that.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.