Closed Bug 919555 Opened 8 years ago Closed 8 years ago
.me][perf] Remove app index duplicacy
Evme indexes Grid Apps which takes over 600k. This should be avoided.
Assignee: nobody → ran
Status: NEW → ASSIGNED
This patch necessary code changes from Bug 911568
This PR takes off over 400k because it removes the need for 2 installed app indexes. In order to test the feature, simply search for the following: * Category - apps in this category will appear ("utilities", "music", "social") * Apps begining with the query (type "m" and marketplace, music, etc. will appear)
blocking-b2g: --- → koi+
Whiteboard: [c= p= s= u=][perf-reviewed] → [c= p= s= u=1.2]
I removed this as a blocker as it's not a huge benefit and Bug 921100 should have brought in a substantial mem save (because of Bug 916060).
Ran - Is this part of the 1.2 e.me changes or is this separate? Trying to figure out if this needs to remain as a koi+ or not.
No it shouldn't. Please change to koi-.
Amir, can you rebase the commit?
Comment on attachment 823994 [details] redirect to PR 13181.html The homescreen code looks good to me! r+
Attachment #823994 - Flags: review?(crdlc) → review+
updated PR, Cristian - changes in page.js and grid.js as we discussed Ran - ready for your r+ now :) Thanks
Comment on attachment 823994 [details] redirect to PR 13181.html Kudos! 500k memory consumption reduction!
Attachment #823994 - Flags: review?(ran) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [c=memory p= s= u=] → [c=memory p= s=2013.11.08 u=]
Clearing nom - this is already part of 1.3, so this doesn't need to be triaged.
blocking-b2g: 1.3? → ---
You need to log in before you can comment on or make changes to this bug.