Comment 2 for bug 1076744

Hey ken,

Two things needs to be fixed IMHO:
* debian/copyright:
- the files under lib/* are GPL2+ and are not really listed in that file, can you please fix that?
* build-dep on vim-common? are you sure? ;)

the rests are more questions:
* debian/xul-ext-webaccounts.postinst:
only firefox needs to be restarted, nothing else?
* how can I test it? like, if I install the extension, I should go to my gmail account (without being logged) and online account will log for me?
* how do you want to deal installing that by default? Seeding the firefox plugin, letting chromium users installing it themselves? adding them as recommends by default? (what about other flavors?). Looking at the js code, it seems the other way aroud: if I login to gmail in a page, it will extract and we'll get the credential stored in addition to chromium in uoa.

* Not related to that directly, but as you are changing the packaging anyway:
- for coherence reason, can you try having all build-deps on top? (likes debhelper, dh_autoreconf, quilt… then upstream build requirement, gcc, vala… and then what is needed for autoreconf, like gnome-common?)

Otherwise, the codes looks good, I didn't see anything dangerous after a look at the javascript.
the chrome manifest is using version 2 if I see correctly! I love it :)