Ticket #2558 (s3b_implementation_finished bug)

Opened 7 years ago

Last modified 7 years ago

[html] Memory leaks in JS and DOM

Reported by: antoni Owned by: antoni
Priority: major Milestone: Y1
Component: uncategorized Version: 2.1
Keywords: Cc:
Category: unknown Effort:
Importance: Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: Design_owners:
Imp._owners: Test_owners:
Analysis_reviewers: Changelog: Changelog
Design_reviewers: Imp._reviewers:
Test_reviewers: Analysis_score: 0
Design_score: 0 Imp._score: 0
Test_score: 0

Description

Inspect memory leaks caused by JS and DOM and try to remove them.

Change History

comment:1 Changed 7 years ago by antoni

  • Status changed from new to s1b_analysis_finished

comment:2 Changed 7 years ago by antoni

  • Status changed from s1b_analysis_finished to s2a_design_started

comment:3 Changed 7 years ago by antoni

  • Status changed from s2a_design_started to s2b_design_finished

comment:4 Changed 7 years ago by antoni

  • Status changed from s2b_design_finished to s2c_design_ok

comment:5 Changed 7 years ago by antoni

  • Owner set to antoni
  • Status changed from s2c_design_ok to s3a_implementation_started

comment:6 Changed 7 years ago by antoni

#2558 : Some patterns used to prevent memory leaks were applied. Most of them were about closure functions. Unfortunately it doesn't decrease memory usage and leaks at all.

Some of these patterns are reviewed at links below:
http://www.javascriptkit.com/javatutors/closuresleak/index3.shtml
http://jibbering.com/faq/notes/closures/
http://www.ibm.com/developerworks/web/library/wa-memleak/

Code was commited at revision [10106].

comment:7 Changed 7 years ago by antoni

Research about memory leak show that console.log in cleanPage method unexpected leak memory. That's why all logs in reader were commented.
These changes were commited in revisions [10117], [10118] and [10119].

comment:8 Changed 7 years ago by antoni

Media objects and their controls construction were optimized to prevent circular memory leaks.

Some console.logs from comments.js were removed.

Code is commited in revision [10123].

comment:9 Changed 7 years ago by antoni

If we cant' swipe because of some error, we set next/previous page content as loaded 5 second later to enable swapping again.

Code is commited in revision [10128].

comment:10 Changed 7 years ago by antoni

  • Status changed from s3a_implementation_started to s3b_implementation_finished

Last attempts to disable updateCallback function on play method for native application wasn't stable so they weren't commited.

Task is closed.

Note: See TracTickets for help on using tickets.