Closed
Bug 1178739
Opened 9 years ago
Closed 5 years ago
Fix HomeProvider eslint errors
Categories
(Firefox for Android Graveyard :: Awesomescreen, defect)
Tracking
(fennec-)
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
fennec | - | --- |
People
(Reporter: sebastian, Unassigned)
References
Details
This error is showing up in the log of Nightly after installing an add-on that adds a home panel. Subsequently the panel remains empty just showing: "No content could be found for this panel."
E [JavaScript Error: "Warning: Sqlite connection 'home.sqlite#5' was not properly closed. Auto-close triggered by garbage collection.
E " {file: "resource://gre/modules/Sqlite.jsm" line: 67}]
Reporter | ||
Comment 1•9 years ago
|
||
This change introduced the regression (bug 1177774):
https://hg.mozilla.org/mozilla-central/rev/8cab4b5abe7f
Michael: The change should have fixed an eslint issue and I'm not sure how this can trigger the error mentioned above. Do you have any idea?
Flags: needinfo?(michael.l.comella)
Reporter | ||
Updated•9 years ago
|
tracking-fennec: --- → ?
Comment 2•9 years ago
|
||
Let's just back out this changeset. Mike and I weren't confident when we made this change, and obviously we missed something!
Comment 3•9 years ago
|
||
I did the backout in bug 1177774. Let's morph this bug to be about fixing the HomeProvider linter errors without breaking things :)
Blocks: 1177774
Summary: JavaScript Error: "Warning: Sqlite connection 'home.sqlite#0' was not properly closed → Fix HomeProvider eslint errors
Comment 4•9 years ago
|
||
We should talk to someone who actually knows about how these jsm contexts work (khuey or fabrice would likely know) to understand what went wrong here.
I also think we should audit the other jsms in /mobile/android/modules, because I suspect we may be doing things wrong in other places, but not running into errors because we're not using sqlite in any of those.
Comment 6•9 years ago
|
||
(In reply to :Margaret Leibovic from comment #4)
> I also think we should audit the other jsms in /mobile/android/modules,
> because I suspect we may be doing things wrong in other places, but not
> running into errors because we're not using sqlite in any of those.
NI self to audit my linter changes.
tracking-fennec: ? → -
Comment 7•9 years ago
|
||
I'll get to this eventually – don't need the NI as it's already on my mind.
Flags: needinfo?(michael.l.comella)
Reporter | ||
Updated•9 years ago
|
Blocks: home-panels
Comment 9•9 years ago
|
||
I ignored this file in bug 1229433 to make `mach eslint mobile` pass.
Blocks: android-jslint
No longer depends on: 1229433
Comment 10•9 years ago
|
||
Mark mentioned this might have something to do with bug 1185578.
Updated•9 years ago
|
Comment 11•7 years ago
|
||
Bug 1378788 is going to enable eslint for HomeProvider.jsm - it is clean now from an ESLint perspective.
Depends on: 1378788
Comment 12•5 years ago
|
||
This file has gone away now, but was probably fixed by bug 1378788.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•