Closed Bug 792062 Opened 12 years ago Closed 12 years ago

Make the tabNavigated notification reusable by the Web Console

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 20

People

(Reporter: msucan, Assigned: msucan)

References

Details

Attachments

(2 files, 1 obsolete file)

We need tabNavigated to fire at the start and end of a location change. For details see bug 768096 comment 36.
Priority: -- → P3
Assignee: nobody → mihai.sucan
Status: NEW → ASSIGNED
Attached patch proposed patch (obsolete) (deleted) — Splinter Review
This is the patch from the pull request. Try run: https://tbpl.mozilla.org/?tree=Try&rev=b1482f94a839 Please let me know if further changes are needed. Thank you!
Attachment #691374 - Flags: review?(past)
Testing Fennec almost everything seems to work OK, except the following message: Connection timeout. Check the Error Console on both ends for potential error messages. Reopen the Web Console to try again. This appears in the web console a few seconds after connecting to Fennec. It also appears after connecting a patched web console to an unpatched Fennec nightly, so I'm guessing the frontend code is at fault. Nothing is present in the browser's error console and I don't see anything relevant in adb logcat, but I might need to flip some prefs there. Any idea where to look next?
Connection timeout message shows only if WCCP_connect doesn't finish. It attaches to a tab and to a console actor. Maybe one of these fail?
Attached file Protocol log (deleted) —
I see both tabAttached and startedListeners responses. And the console seems to work fine despite that message (autocompletion, evaluation, etc.).
Thanks. I'll look into this. Working on building Fennec here.
Attached patch fixed patch (deleted) — Splinter Review
Fixed it. Should work now with Fennec. Tested on my system with B2G Desktop and Fennec on Android (emulator). I missed a callback. Please double-check and also verify that the debugger is unaffected. I'd like to land this patch ASAP. Thank you!
Attachment #691374 - Attachment is obsolete: true
Attachment #691374 - Flags: review?(past)
Attachment #693063 - Flags: review?(past)
Comment on attachment 693063 [details] [diff] [review] fixed patch Review of attachment 693063 [details] [diff] [review]: ----------------------------------------------------------------- Yep, seems fixed now. Let's land this puppy!
Attachment #693063 - Flags: review?(past) → review+
Whiteboard: [fixed-in-fx-team]
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 20
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: