ma première extension, Boox

[martin]

En effet, merci, ça marche pour Libé et Le Soir, mais rien à faire pour Le Figaro, ça reste décalé.

arf, effectivement en ce moment précis oui.
Il y a une entrée du fil qui a un titre vide, il se trouve que Firefox zappe cette entrée en garnissant le MP dynamique... Boox lui en tient compte et se gourre dans les incrémentations .
On va espérer pour l'instant que cette "erreur" de ce fil est exceptionnelle, et donc que ce sera résolu lorsque de nouvelles entrées remplaceront les anciennes. Sinon, il faudra que je trouve une parade...
A suivre de nouveau...
(merci FF_Olivier )

[martin]

nouvelle version 1.0.1.0

corrige un bug, le lecteur de fil était cassé avec certaines compilations de Firefox, comme celle d'ubuntu.

La mise à jour automatique est en place.

[martin]

nouvelle version 1.0.2.0

mise à jour pour coller aux évolutions de FeedProcessor, + ajout de la locale zw-TW.

La mise à jour automatique est en place.

[The RedBurn]

Salut,

Venant de réactiver Leak Monitor, j'ai remarqué qu'elle indiquait une fuite de mémoire liée à Boox lors de l'ouverture/fermeture du panneau latéral des marque-pages.

[martin]

Venant de réactiver Leak Monitor, j'ai remarqué qu'elle indiquait une fuite de mémoire liée à Boox lors de l'ouverture/fermeture du panneau latéral des marque-pages.

bonsoir, et merci.
allons bon, j'ai également Leak Monitor d'activé en ce moment, et aucun soucis .
aurais tu la gentillesse de me dire ta config (OS), et ta liste d'extensions (si c'est déclenché par une interaction) ?

[The RedBurn]

Windows XP avec les extensions suivantes activées :
* Adblock Filterset.G Updater 0.3.1.0
* Adblock Plus 0.7.5.1
* Adblock Plus: Element Hiding Helper 1.0.1
* Always Remember Password 0.6
* Bookmark Backup 0.4.6
* Bookmark Duplicate Detector 0.6.3
* Boox 1.0.2.0
* CuteMenus2 2.0
* Dictionary Switcher 0.8
* Dictionnaire MySpell en Français 1.0.1
* DOM Inspector 1.8.1.6
* Extended Link Properties 1.2.3
* Firebug 1.05
* Forecastfox l10n 0.7.2007050801
* Frequence3 Top/Flop 1.0.0.0
* Googlebar Lite 4.6.3
* IE View 1.3.3
* Java Console 6.0.02
* Linkification 1.3.3
* Live HTTP Headers 0.13.1
* Long Titles 1.3
* Minimeter 1.0.7.7
* MR Tech Local Install 5.3.2.5
* OpenBook 1.3.4.2
* Remember Mismatched Domains 1.4.3
* Restore Scroll Position 0.4
* Session Manager 0.5.4.1
* SmoothWheel 0.44.9.20070224
* Tab Killer 1.1.2007032801
* Talkback 2.0.0.6
* United States English Dictionary 2.0.0.6
* View Dependencies 0.3.2.2

Il ne te reste plus qu'à les tester une par une.

Bon allez, pour t'épargner la peine, j'ai testé en les enlevant toutes sauf Leak Monitor. Verdict : il indique toujours une fuite.

[martin]

merci.
Il me faudra néanmoins les essayer toutes.

La je viens de vérifier sur un profil neuf et sur XP, avec Boox et Leak Monitor, et quelques flux dans les bookmarks, pas de fuites...

Merci encore de l'info, j'essaierai un peu plus demain (si je trouve le temps).

[The RedBurn]

Je n'aurais pas dû te donner la liste.
D'un autre côté, certaines te plairont peut-être. Boox, par exemple, n'est pas mal.
Mais étant donné que j'ai testé en supprimant les extensions du profil, je ne pense pas que le problème vienne de là (je vais tester avec un nouveau profil).
Voici le rapport de fuite :

Code : Tout sélectionner

Leaks in window 0x26df798:
[+] [leaked object] (2e6f8b0) = [Object]
 [+] prefs (2fc5620) = [XPCWrappedNative_NoHelper]
  [ ] QueryInterface (2fc5618) = [Function]
  [ ] addObserver (2fc5610) = [Function]
  [ ] root = true
  [+] getPrefType (2136058) = [Function]
   [ ] prototype (250d9e0) = [Object]
  [+] getBoolPref (2135260) = [Function]
   [ ] prototype (250da20) = [Object]
  [+] setBoolPref (2135248) = [Function]
   [ ] prototype (250da58) = [Object]
  [+] getCharPref (240f3a0) = [Function]
   [ ] prototype (250da90) = [Object]
  [+] setCharPref (240f368) = [Function]
   [ ] prototype (250db00) = [Object]
  [+] getIntPref (24cfd88) = [Function]
   [ ] prototype (250db10) = [Object]
  [+] setIntPref (24cf870) = [Function]
   [ ] prototype (250db38) = [Object]
  [+] getComplexValue (250f3d0) = [Function]
   [ ] prototype (250db68) = [Object]
  [+] setComplexValue (250f178) = [Function]
   [ ] prototype (250dba8) = [Object]
  [+] clearUserPref (250ec38) = [Function]
   [ ] prototype (250dc10) = [Object]
  [+] lockPref (250e880) = [Function]
   [ ] prototype (250dc28) = [Object]
  [+] prefHasUserValue (250e570) = [Function]
   [ ] prototype (250dc70) = [Object]
  [+] prefIsLocked (250e070) = [Function]
   [ ] prototype (250dc80) = [Object]
  [+] unlockPref (250e038) = [Function]
   [ ] prototype (250dcb8) = [Object]
  [+] deleteBranch (250de90) = [Function]
   [ ] prototype (250dd10) = [Object]
  [+] getChildList (250de60) = [Function]
   [ ] prototype (250dd38) = [Object]
  [+] resetBranch (250de38) = [Function]
   [ ] prototype (250dd68) = [Object]
  [ ] PREF_INVALID = 0
  [ ] PREF_STRING = 32
  [ ] PREF_INT = 64
  [ ] PREF_BOOL = 128
  [+] removeObserver (250de00) = [Function]
   [ ] prototype (250ddf0) = [Object]
 [+] startup (2e6f898, chrome://boox/content/booxBookmarksPanel.js, 630-633) = [Function]
  [ ] prototype (2136b60) = [Object]
 [+] shutdown (2e6f820, chrome://boox/content/booxBookmarksPanel.js, 635-636) = [Function]
  [ ] prototype (2136bc8) = [Object]
 [+] observe (2e6f7e8, chrome://boox/content/booxBookmarksPanel.js, 638-672) = [Function]
  [ ] prototype (2084d50) = [Object]
[+] [leaked object] (2e70230) = [Object]
 [+] booxPrefs (2fc5060) = [XPCWrappedNative_NoHelper]
  [ ] QueryInterface (2fc5058) = [Function]
  [ ] addObserver (2fc5050) = [Function]
  [ ] root = true
  [+] getPrefType (250d840) = [Function]
   [ ] prototype (25b9768) = [Object]
  [+] getBoolPref (250d838) = [Function]
   [ ] prototype (25b9770) = [Object]
  [+] setBoolPref (250d830) = [Function]
   [ ] prototype (25b9778) = [Object]
  [+] getCharPref (250d828) = [Function]
   [ ] prototype (25b9798) = [Object]
  [+] setCharPref (250d820) = [Function]
   [ ] prototype (25b97a0) = [Object]
  [+] getIntPref (250d818) = [Function]
   [ ] prototype (25b97d0) = [Object]
  [+] setIntPref (250d810) = [Function]
   [ ] prototype (25b97d8) = [Object]
  [+] getComplexValue (250d808) = [Function]
   [ ] prototype (25b97f8) = [Object]
  [+] setComplexValue (250d7f8) = [Function]
   [ ] prototype (250d6e0) = [Object]
  [+] clearUserPref (250d7f0) = [Function]
   [ ] prototype (250d6e8) = [Object]
  [+] lockPref (250d7c0) = [Function]
   [ ] prototype (250d6f8) = [Object]
  [+] prefHasUserValue (250d7b8) = [Function]
   [ ] prototype (250d700) = [Object]
  [+] prefIsLocked (250d7b0) = [Function]
   [ ] prototype (250d708) = [Object]
  [+] unlockPref (250d7a8) = [Function]
   [ ] prototype (250d710) = [Object]
  [+] deleteBranch (250d7a0) = [Function]
   [ ] prototype (250d718) = [Object]
  [+] getChildList (250d790) = [Function]
   [ ] prototype (250d748) = [Object]
  [+] resetBranch (250d788) = [Function]
   [ ] prototype (250d770) = [Object]
  [ ] PREF_INVALID = 0
  [ ] PREF_STRING = 32
  [ ] PREF_INT = 64
  [ ] PREF_BOOL = 128
  [+] removeObserver (250d780) = [Function]
   [ ] prototype (250d778) = [Object]
 [+] startup (2e70210, chrome://boox/content/booxBookmarksPanel.js, 42-45) = [Function]
  [ ] prototype (250d850) = [Object]
 [+] shutdown (2e70200, chrome://boox/content/booxBookmarksPanel.js, 47-48) = [Function]
  [ ] prototype (250d860) = [Object]
 [+] observe (2e701f0, chrome://boox/content/booxBookmarksPanel.js, 50-67) = [Function]
  [ ] prototype (250d870) = [Object]

[The RedBurn]

Ça y est, j'ai trouvé ce qui déclenche le problème : c'est lorsqu'on désactive le stylage des marque-pages dynamiques dans le panneau (latéral) des marque-pages.

[martin]

magnifique The RedBurn , çà c'est du rapport de bug !

Ok, reproduit, pisté, solution trouvée.
Je package, extensionne et xpéise çà au plus vite.

[martin]

voilà version 1.0.3.0 en ligne, corrigeant le bug. Encore merci The RedBurn .

Là , j'ai plus le temps, je met la mise à jour automatique en place ce soir, ainsi que la soumission sur AMO. En attendant vous pouvez installer cette version sur mon site.

[The RedBurn]

De rien.

En parlant, de stylage des marque-pages dynamiques, si tu ne sais plus quoi faire, n'hésite pas à ajouter la possibilité de marquer les entrées lues via la favicon (ou une icône générique si le site n'en a pas). Car si tout le monde n'aime pas la mise en gras des entrées non lues, la mémorisation des entrées lues doit sûrement être très appréciée.

J'oubliais de dire : bravo pour la rapidité de correction du problème.

[martin]

bon finalement j'ai pris le temps de mettre la mise à jour automatique en place , reste plus que la soumission sur AMO que je ferai ce soir.

En parlant, de stylage des marque-pages dynamiques, si tu ne sais plus quoi faire, n'hésite pas à ajouter la possibilité de marquer les entrées lues via la favicon (ou une icône générique si le site n'en a pas). Car si tout le monde n'aime pas la mise en gras des entrées non lues, la mémorisation des entrées lues doit sûrement être très appréciée.

Tu peux préciser ? Pour l'instant on peut marquer comme lu tout le contenu d'un marque-page dynamique via un clic-droit sur celui-ci, c'est pas suffisant ?

[The RedBurn]

En parlant, de stylage des marque-pages dynamiques, si tu ne sais plus quoi faire, n'hésite pas à ajouter la possibilité de marquer les entrées lues via la favicon (ou une icône générique si le site n'en a pas). Car si tout le monde n'aime pas la mise en gras des entrées non lues, la mémorisation des entrées lues doit sûrement être très appréciée.

Tu peux préciser ? Pour l'instant on peut marquer comme lu tout le contenu d'un marque-page dynamique via un clic-droit sur celui-ci, c'est pas suffisant ?

Non, je parle du fait que quand on active le stylage des marque-pages dynamiques, ceux lus sont différenciés par le fait qu'ils ne sont pas en gras. Mais quand on désactive le stylage...
D'où ma suggestion d'utiliser la favicon (comme le fait déjà Firefox pour les sites qui en ont une, mais sans le mémoriser entre les sessions) pour la différenciation.
Le mot "marquer" est ambigu.

[martin]

nouvelle version soumise sur AMO.

Ah ok The RedBurn, tu suggères une favicon particulière pour différencier les entrées non lues, c'est çà ?

En principe, il n'y aura pas pour l'instant de nouvelles fonctionnalités prévues pour Boox (mais je peux retenir les idées qui me plaisent pour plus tard ). La suite, quand je trouverai le temps, est de la réécrire pour Firefox 3, l'extension étant fortement impactée par cette nouvelle version et la mise en place de Places.

Au passage, des fois que çà aide d'autres developpeurs, la fuite de mémoire était due au faite que supprimer un observateur qui n'a pas été enregistré préalablement (en plus précis appliquer la méthode removeObserver d'un objet nsIObserverService si aucun observateur n'a été enregistré auparavant par addObserver) plante la suite du script. Je sais pas trop si on doit considérer çà comme un bug (que je devrai alors rapporté), c'est logique, mais en même temps ça devrait pas planter pour autant...