Closed Bug 1136151 Opened 10 years ago Closed 6 years ago

Breakpoint in contentAreaClick not hit in new windows

Categories

(DevTools :: Debugger, defect)

All
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: Gijs, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

This worked a few months ago (bug 883342) but is broken now. STR: 1. open Firefox + browser debugger 2. go to www.mozilla.org 3. in the browser debugger, open chrome://browser/content/browser.js 4. find 'function contentAreaClick' 5. set a breakpoint on the first line 6. click a link on the www.mozilla.org homepage --> the breakpoint gets hit 7. open a new window and open www.mozilla.org again (alt: drag the tab down to create a new window with the same page) 8. click a link again --> the breakpoint does not get hit
Product: Firefox → DevTools

I was testing this and my BP didn't hit at step #6

@Gijs: do you see the same?

Honza

Flags: needinfo?(gijskruitbosch+bugs)

(In reply to Jan Honza Odvarko [:Honza] (always need-info? me) from comment #1)

I was testing this and my BP didn't hit at step #6

@Gijs: do you see the same?

Honza

Of course, because this was 4 years ago and now we have e10s, so contentAreaClick isn't called in the parent process anymore, we use ContentClick.jsm or whatever it's called, on the content process.

https://searchfox.org/mozilla-central/source/browser/base/content/browser.js#1491-1495

The same bug probably might still exist in other code that does get hit in the e10s case.

Flags: needinfo?(gijskruitbosch+bugs)

I was testing this with the following STRs:

  1. open Firefox + browser debugger
  2. go to www.mozilla.org
  3. in the browser debugger, open resource://devtools/client/netmonitor/src/app.js
  4. set a breakpoint on the first line in bootstrap function (line 33 atm)
  5. Open DevTools select the Network panel
    --> the breakpoint gets hit
  6. open a new window and DevTools + Network panel again
    --> the breakpoint does get hit again

I was also checking some others code (including chrome://), but didn't see the problem.

Closing

@Gijs, please reopen if you can repro that!

Honza

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.