| 5319190a069ce6657577718a72e645dad45be0af |
|
31-May-2012 |
xqian6 <xi.qian@intel.com> |
Fix memory leak caused by CSSRuleList wrapper
Accessing cssrulelist in javascript will create a new
CSSRuleList object and wrapper JS object. The wrapper
JS object will be added into hidden array in parent JS
object which is alive during the whole execution. Thus
memory leak happens (CSSRuleList in webkit, wrapper object
and weak global handle in v8).
Cherry pick 2 patches from upstream solves the problem:
http://trac.webkit.org/changeset/90949
This patch changes hidden reference from array to named
property. So new wrapper will replace old wrappper. But the
memory leak still exists because the CSSRuleList wrapper
will be added into an object group of current document. So
they will still be alive during execution.
http://trac.webkit.org/changeset/91256
This patch avoids to adding CSSRuleList wrapper into document
object group. Combined with the first patch, it can resolve
the memory leak problem.
Change-Id: Icb523db52963726f27b6c02596822cfb6e8d5049
Author: Vitaly Repeshko <vitalyr@chromium.org>
Signed-off-by: Xi Qian <xi.qian@intel.com>
Signed-off-by: Shuo Gao <shuo.gao@intel.com>
Signed-off-by: Bruce Beare <bruce.j.beare@intel.com>
Signed-off-by: Jack Ren <jack.ren@intel.com>
Author-tracking-BZ: 32630
/external/webkit/Source/WebCore/bindings/v8/V8DOMWrapper.cpp
|
| 574543de6f562b52e4e06c8b493c76dea160473f |
|
06-Jun-2011 |
Ben Murdoch <benm@google.com> |
Merge WebKit at r74325: Build fix.
Cherry pick of upstream patch http://trac.webkit.org/changeset/87424
Change-Id: I6546831e7cecf11bbfd194f0620c56f293b56f2d
/external/webkit/Source/WebCore/bindings/v8/V8DOMWrapper.cpp
|
| 2daae5fd11344eaa88a0d92b0f6d65f8d2255c00 |
|
02-Jun-2011 |
Ben Murdoch <benm@google.com> |
Merge WebKit at r84325: Initial merge by git.
Change-Id: Ic1a909300ecc0a13ddc6b4e784371d2ac6e3d59b
/external/webkit/Source/WebCore/bindings/v8/V8DOMWrapper.cpp
|
| 2fc2651226baac27029e38c9d6ef883fa32084db |
|
18-May-2011 |
Steve Block <steveblock@google.com> |
Merge WebKit at r78450: Initial merge by git.
Change-Id: I6d3e5f1f868ec266a0aafdef66182ddc3f265dc1
/external/webkit/Source/WebCore/bindings/v8/V8DOMWrapper.cpp
|
| cad810f21b803229eb11403f9209855525a25d57 |
|
06-May-2011 |
Steve Block <steveblock@google.com> |
Merge WebKit at r75315: Initial merge by git.
Change-Id: I570314b346ce101c935ed22a626b48c2af266b84
/external/webkit/Source/WebCore/bindings/v8/V8DOMWrapper.cpp
|