Closed Bug 1251752 Opened 4 years ago Closed 4 years ago

logging: add tree specific methods

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: surkov, Assigned: surkov)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
No description provided.
Attachment #8724250 - Flags: review?(yzenevich)
Comment on attachment 8724250 [details] [diff] [review]
patch

Review of attachment 8724250 [details] [diff] [review]:
-----------------------------------------------------------------

::: accessible/base/Logging.cpp
@@ +830,5 @@
> +    printf(", document node: %p\n", static_cast<void*>(node));
> +  }
> +  else if (node->IsNodeOfType(nsINode::eTEXT)) {
> +    printf(", text node: %p\n", static_cast<void*>(node));
> +    return;

this return does not looks like it's necessary
Attachment #8724250 - Flags: review?(yzenevich) → review+
https://hg.mozilla.org/mozilla-central/rev/72913e1afccb
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Assignee: nobody → surkov.alexander
You need to log in before you can comment on or make changes to this bug.