959445 - add lldb summaries for nsIAtoms, nsTextNodes and nsTextFragments showing their text content

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Cameron McCormack (:heycam)]

Attachment: patch

Comment 1

[Cameron McCormack (:heycam)]

Attachment: patch v2

Make the string formatting function more lenient about the types it accepts, so we can print nsAStrings etc. later more easily.

Comment 2

[Cameron McCormack (:heycam)]

Attachment: patch v2.1

Sorry one more tweak.

Comment 3

[(no longer active)]

Comment on attachment 8359584 [details] [diff] [review]
patch v2.1

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

This is pretty neat, I must say!

::: python/lldbutils/lldbutils/content.py
@@ +9,5 @@
> +        field = "m2b"
> +    else:
> +        field = "m1b"
> +    ptr = content_union.GetChildMemberWithName(field)
> +    return utils.format_string(ptr, min(length, 100))

Hmm, I don't think we should clamp the string length like this, please remove the clamping code.

Comment 4

[Cameron McCormack (:heycam)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/bb7af8904cac

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/bb7af8904cac

Comment 6

[[e-mail]]

Comment on attachment 8359584 [details] [diff] [review]
patch v2.1

>+    elif c == 0x07:
>+        return "\\a"
>+    elif c == 0x08:
>+        return "\\b"
>+    elif c == 0x0c:
>+        return "\\f"
>+    elif c == 0x0a:
>+        return "\\n"
>+    elif c == 0x0d:
>+        return "\\r"
>+    elif c == 0x09:
>+        return "\\t"
>+    elif c == 0x0b:
>+        return "\\v"
I would have thought numerical order would make more sense than alphabetical order of special character.

>+    elif c < 0x20 or c >= 0x80 and c <= 0xff:
>+        return "\\x%02x" % c
IIRC 0x7f is also unprintable.

>+    elif c >= 0x0100:
If you swap the conditions then you can write
elif c >= 0x0100:
    ...
elif c < 0x20 or c >= 0x7f:
    ...
else:
    etc.

>+    i = 0
>+    terminated = False
>+    while i < length:
>+        c = lldb_value.CreateValueFromAddress("x", ptr + i * size, char_type).GetValueAsUnsigned(0) & mask
>+        if c == 0:
>+            terminated = True
Strings are always the exact provided length; embedded nulls are completely legal, in fact you will only hit this condition in such a case. It's also unsafe to read the length character to check whether the string is terminated; you can look at the flags if you really want to be sure, but it's certainly wrong to add an ellipsis in any event.

Comment 7

[Cameron McCormack (:heycam)]

Neil, can you open a separate bug for these things?