"Top sites" now includes thumbnails of the most visited sites, so the empty screen for that page will never be displayed. We should remove the resources and references to those resources.

this sounds like it would be a good first bug for me. java is my strongest language so it would be a good intro to development for fennec. would you be happy to mentor me on this as my first bug fix when i get my build environment set up?

Hi basscase, I actually completely forgot I had filed this bug and ended up writing a patch for it in another bug - if you'd like to look at the patch, I'll be uploading it here. I apologize for the confusion on my part - there another bug that you think you'd like to try out? And feel free to drop into #mobile on Mozilla's irc server: irc.mozilla.org - we're happy to answer any questions you have there!

Moved the patch from bug 919230, because I forgot I filed this bug.

Comment on attachment 809973 [details] [diff] [review] 1-remove-empty-topsites Review of attachment 809973 [details] [diff] [review]: ----------------------------------------------------------------- Nice :D

(oops, clipboard fail) https://hg.mozilla.org/integration/fx-team/rev/e6af0f7d99da

[Approval Request Comment] Bug caused by (feature/regressing bug #): bug 917394 User impact if declined: bug 920317 depends on this patch Testing completed (on m-c, etc.): on m-c for a few weeks Risk to taking this patch (and alternatives if risky): low, code removal of xml and resources String or IDL/UUID changes made by this patch: none (leaving in strings that are removed in 27)

(Updated the patch message to match this bug) [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 917394 User impact if declined: bug 920317 depends on this patch Testing completed (on m-c, etc.): on m-c for a few weeks Risk to taking this patch (and alternatives if risky): low, code removal of xml and resources String or IDL/UUID changes made by this patch: none (leaving in strings that are removed in 27)