Closed Bug 1322083 Opened 7 years ago Closed 7 years ago

Reps with a more link shows -3 more...

Categories

(DevTools :: Shared Components, defect)

defect
Not set
normal

Tracking

(firefox53 fixed)

VERIFIED FIXED
Firefox 53
Tracking Status
firefox53 --- fixed

People

(Reporter: ntim, Assigned: nchevobbe)

Details

Attachments

(2 files)

Attached image Screenshot
It would be nice if we had a test that caught this.
Thanks for reporting this Tim.

We dealt with similar thing in Bug 1305981, and we added some tests there.
The problem is that it heavily depends on what's in the packet, and can't be easily testable before we know what the packet have or do not have.
Here, it seems that the applicationCache object does not have any "object length" property we're looking for in the Rep.
What's weird is that the applicationCache object in the DOM Panel only show the class name in v50. Something must have changed because we show the internals now.

I'll look for that change and will find a way to display this correctly.

We can find a way to test all the window properties and make sure that if they do show a "more..." label, it is > 0. But at first glance I'm afraid it might take quite some time (there's more than 250 properties on the window object).
Assignee: nobody → chevobbe.nicolas
Comment on attachment 8819655 [details]
Bug 1322083 - Fix Reps with safeGetterValues;

https://reviewboard.mozilla.org/r/99352/#review99624

Works great, thanks! r+ if try is green.
Attachment #8819655 - Flags: review?(ntim.bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/114273f3f712
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
I have reproduced this bug with Nightly 53.0a1 (2016-12-04) (32-bit) on Windows 8.1 (64-bit)

This bug's fix is verified on Latest Nightly 53.0a1 (2016-12-28) (32-bit).

Build ID : 20161228030213
User Agent : Mozilla/5.0 (Windows NT 6.3; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0

[bugday-20161228]
The bug is verified on Mozilla Nighty version. [bugday-20160104]
Status: RESOLVED → VERIFIED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: