Closed Bug 842395 Opened 9 years ago Closed 9 years ago
Clients Available() from Tabs Accessor
We aren't using this method anywhere.
Attachment #715256 - Flags: review?(mark.finkle)
I also removed the unused OnClientsAvailableListener interface locally.
Attachment #715256 - Flags: review?(mark.finkle) → review+
Have you talked to rnewman about whether or not sync needs this?
(In reply to :Margaret Leibovic from comment #2) > Have you talked to rnewman about whether or not sync needs this? I didn't see it used in the sync code either, but I guess it couldn't hurt to ask.
Looking through the history here a bit more, I think you're right that we don't need this anymore. I had just remembered working with rnewman on TabsAccessor, so I recalled some sync dependency. Never hurts to ask through! :)
Thanks, Margaret, Brian! This used to be used in TabsTray to hide or show the remote tabs UI (Bug 734425). Looks like it's dead code now. Yay for shrinkage!
Status: NEW → ASSIGNED
Depends on: 734425
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.