Closed Bug 770882 Opened 7 years ago Closed 7 years ago

resizer.xml visible in the debugger

Categories

(DevTools :: Debugger, defect, P2)

x86
All
defect

Tracking

(firefox15 affected, firefox16- fixed)

RESOLVED FIXED
Firefox 17
Tracking Status
firefox15 --- affected
firefox16 - fixed

People

(Reporter: paul, Assigned: past)

References

Details

Attachments

(3 files)

Attached image screenshot
STR:
- open about:blank
- open the debugger
- open the Responsive Mode
I can't reproduce. Maybe fixed already.
STR work for me on Win 7 with the latest Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0 (Built from http://hg.mozilla.org/mozilla-central/rev/87db9617a885).

After opening the responsive view, the debugger loads resizer.xml:

<?xml version="1.0"?>
<!-- This Source Code Form is subject to the terms of the Mozilla Public
   - License, v. 2.0. If a copy of the MPL was not distributed with this
   - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->


<bindings id="resizerBindings"
   xmlns="http://www.mozilla.org/xbl">

  <binding id="resizer">
    <resources>
      <stylesheet src="chrome://global/skin/resizer.css"/>
    </resources>
    <implementation>
      <constructor>
      <![CDATA[
        // don't do this for viewport resizers; causes a crash related to
        // bugs 563665 and 581536 otherwise
        if (this.parentNode == this.ownerDocument.documentElement)
          return;

        // if the direction is rtl, set the rtl attribute so that the
        // stylesheet can use this to make the cursor appear properly
        var direction = window.getComputedStyle(this, "").direction;
        if (direction == "rtl") {
          this.setAttribute("rtl", "true");
        }
      ]]>
      </constructor>
    </implementation>
  </binding>

</bindings>
There's a screenshot in bug 767390 that also shows resizer.xml in the Debugger.
Priority: -- → P2
Assignee: nobody → past
Status: NEW → ASSIGNED
Attached patch Working patchSplinter Review
OK, so the fact that it took me so long to see what is wrong here, is rather embarrassing. Apparently though a holiday can seriously clear your mind!
Attachment #653413 - Flags: review?(rcampbell)
Comment on attachment 653413 [details] [diff] [review]
Working patch

Seem to be able to trigger this with a reload while the debugger's active. This should fix.
Attachment #653413 - Flags: review?(rcampbell) → review+
doesn't apply to beta, would like to include it in aurora and beta as it's a pretty basic patch that pre-empts some user confusion.
Attached patch Patch for betaSplinter Review
This applies cleanly to beta.
Attachment #654138 - Flags: review?(rcampbell)
Comment on attachment 654138 [details] [diff] [review]
Patch for beta

we missed the boat on beta. Sorry.
Attachment #654138 - Flags: review?(rcampbell) → review-
Comment on attachment 653413 [details] [diff] [review]
Working patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature. 
User impact if declined: users may see mysterious system scripts in the debugger during reloads.
Testing completed (on m-c, etc.): on m-c, local
Risk to taking this patch (and alternatives if risky): negligible.
String or UUID changes made by this patch:  none

I think this should apply fairly easily on aurora, requesting approval.
Attachment #653413 - Flags: approval-mozilla-aurora?
Comment on attachment 653413 [details] [diff] [review]
Working patch

I don't see the need to track this for 16, we can approve for Aurora though.
Attachment #653413 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/df17d6a5a330
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 17
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.