Closed
Bug 615846
Opened 15 years ago
Closed 15 years ago
correct name/test_markup.html, extend events debugging in mochitests
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla2.0b8
People
(Reporter: surkov, Assigned: surkov)
Details
(Keywords: access)
Attachments
(1 file, 1 obsolete file)
18.12 KB,
patch
|
fherrera
:
review+
|
Details | Diff | Splinter Review |
patch is against the trunk
Attachment #494358 -
Flags: review?(fherrera)
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•15 years ago
|
||
a bit better version
Attachment #494358 -
Attachment is obsolete: true
Attachment #494390 -
Flags: review?(fherrera)
Attachment #494358 -
Flags: review?(fherrera)
Comment 2•15 years ago
|
||
Comment on attachment 494390 [details] [diff] [review]
patch2
Nice.
It would be nice to add also:
var gA11yEventDumpToAppConsole = false;
...
var consoleService = Components.classes["@mozilla.org/consoleservice;1"]
.getService(Components.interfaces.nsIConsoleService);
...
logToAppConsole: function logger_logToAppConsole(aInfo)
{
if (gA11yEventDumpToAppConsole)
consoleService.logStringMessage("events: " + aInfo);
}
but it's up to you as we already have two dumping methods.
r=me
Attachment #494390 -
Flags: review?(fherrera) → review+
Assignee | ||
Comment 3•15 years ago
|
||
landed with comment addressed - http://hg.mozilla.org/mozilla-central/rev/d25494ae4c3b
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in
before you can comment on or make changes to this bug.
Description
•