Closed
Bug 659777
Opened 13 years ago
Closed 13 years ago
Cannot access remote/desktop tabs: "Svc.Private is undefined"
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
VERIFIED
FIXED
mozilla6
Tracking | Status | |
---|---|---|
firefox5 | --- | unaffected |
firefox6 | --- | fixed |
fennec | 6+ | --- |
People
(Reporter: mbrubeck, Assigned: mbrubeck)
References
Details
(Keywords: regression, Whiteboard: [verified in services])
Attachments
(2 files)
3.44 KB,
patch
|
philikon
:
review+
mfinkle
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
43.57 KB,
image/png
|
Details |
Steps to reproduce: 1. Connect Fennec to a sync account. 2. Open the "Desktop" tab on the awesome screen. Expected results: List of remote tabs is displayed. Actual results: No remote tabs are displayed. Console error: "Svc.Private is undefined" resource://services-sync/engines/tab.js Line 219 This is reproducible in both desktop and Android, in both Nightly (7.0a1) and Aurora (6.0a2).
Assignee | ||
Comment 1•13 years ago
|
||
Regression from bug 648364 - there is no private browsing service in Fennec, and Svc.Private no longer falls back on a dummy implementation.
Blocks: 648364
Assignee | ||
Updated•13 years ago
|
Assignee | ||
Comment 2•13 years ago
|
||
The problem is that nsIPrivateBrowsingService is not implemented in Fennec, and Sync no longer falls back on a dummy implementation. This patch modifies the Svc.Private getter to return undefined always (instead of throwing an exception the first time and returning undefined later), and checks the existence of Svc.Private before trying to use it. Alternately, you could go back to using a dummy service for Svc.Private, or we could implement a dummy service in Fennec.
Attachment #535220 -
Flags: review?(philipp)
Comment 3•13 years ago
|
||
Comment on attachment 535220 [details] [diff] [review] patch >+XPCOMUtils.defineLazyGetter(Svc, "Private", function() { >+ try { >+ return Cc["@mozilla.org/privatebrowsing;1"].getService(Ci["nsIPrivateBrowsingService"]); >+ } catch (e) { >+ return undefined; >+ } >+}); Please add a comment above this stanza to explain why we're doing this lazy getter the long way round. Looks good otherwise. Please commit on services-central, not m-c. I can also land it for you if you prefer.
Attachment #535220 -
Flags: review?(philipp) → review+
Comment 4•13 years ago
|
||
Matt, btw, do you think there's a way to unit test this? I want to make sure we only make mistakes once ;) (We really need TPS (nee Crossweave) tests to run against an actual Fennec instance for the 'mobile' test run, then we would actually have caught this.)
Assignee | ||
Comment 5•13 years ago
|
||
http://hg.mozilla.org/services/services-central/rev/054adb76530d We can try to add some Fennec browser-chrome tests to catch sync bugs, until we are able to get the xpcshell tests running for mobile.
Flags: in-testsuite?
Whiteboard: [fixed in services]
Assignee | ||
Comment 6•13 years ago
|
||
Comment on attachment 535220 [details] [diff] [review] patch Requesting approval-mozilla-aurora. This is a regression compared to Firefox 5 that affects Firefox 6 and breaks major functionality in Fennec. The fix is low risk and localized (adding a single try-catch block and null guards). The alternate to this fix is to back out bug 648364 which was a large refactoring (over 1000 lines changed). Since other changes to the code have landed on top of that refactoring, it probably cannot be backed out without manual conflict resolution.
Attachment #535220 -
Flags: approval-mozilla-aurora?
Comment 7•13 years ago
|
||
Comment on attachment 535220 [details] [diff] [review] patch Looks safe
Attachment #535220 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 8•13 years ago
|
||
Pushed to Aurora for Firefox 6: http://hg.mozilla.org/releases/mozilla-aurora/rev/c2a6c74bcfa4 This is not yet fixed on trunk. It will land in the next merge from services-central to mozilla-central.
Target Milestone: --- → Firefox 6
Updated•13 years ago
|
Component: General → Firefox Sync: Backend
Product: Fennec → Mozilla Services
QA Contact: general → sync-backend
Target Milestone: Firefox 6 → mozilla6
Version: Trunk → unspecified
Comment 9•13 years ago
|
||
Updated•13 years ago
|
tracking-fennec: ? → 6+
Comment 11•13 years ago
|
||
STRs for QA: Tabs from Other Computers should work on Fennec again now (was briefly broken on Aurora and Nightlies).
Comment 12•13 years ago
|
||
WFM: Verified Fixed on Aurora Mozilla/5.0 (Android; Linux armv7l; rv:6.0a2) Gecko/20110531 Firefox/6.0a2 Fennec/6.0a2
Whiteboard: [fixed in services] → [verified in services]
Comment 13•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/054adb76530d
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 15•13 years ago
|
||
Verified fixed on Firefox 6 Beta 2: Mozilla /5.0 (Android;Linux armv7l;rv:6.0) Gecko/20110713 Firefox/6.0 Fennec/6.0 Device: LG Optimus 2X (Android 2.2)
Status: RESOLVED → VERIFIED
Assignee | ||
Updated•12 years ago
|
Flags: in-testsuite?
Updated•5 years ago
|
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•