Attachment 371647 Details for Bug 198666 – Patch

[patch] Patch

bug-198666-20190608133043.patch (text/plain), 1.92 KB, created by cathiechen on 2019-06-07 22:30:44 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: cathiechen

Created: 2019-06-07 22:30:44 PDT

Size: 1.92 KB

patch

obsolete

>Subversion Revision: 246225
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index dcbc10915ea79df6893448a18c16acbbbb48cb81..06be26c50d090e4f561b185dc8dbcd269f76e085 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,16 @@
>+2019-06-07  Cathie Chen  <cathiechen@igalia.com>
>+
>+        resize-observer/element-leak.html is a flaky failure
>+        https://bugs.webkit.org/show_bug.cgi?id=198666
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        It takes very long time to construct iframes which create and remove 1000 DIVs.
>+        We don't need 1000 element to confirm the GC problem. So in order to reduce the fake alarm.
>+        Change the number of DIV to 200.
>+
>+        * resize-observer/resources/element-leak-frame.html:
>+
> 2019-06-07  Truitt Savell  <tsavell@apple.com>
> 
>         Unreviewed, rolling out r246138.
>diff --git a/LayoutTests/resize-observer/resources/element-leak-frame.html b/LayoutTests/resize-observer/resources/element-leak-frame.html
>index b162d44d424f3a009bb8f31ee350cd48504f4e10..6da757f16c77dd2a2ea4a0e86bbd308c5f3daca4 100644
>--- a/LayoutTests/resize-observer/resources/element-leak-frame.html
>+++ b/LayoutTests/resize-observer/resources/element-leak-frame.html
>@@ -3,7 +3,7 @@
> <script src="../../resources/gc.js"></script>
> <script type="text/javascript">
> 
>-const targetCount = 1000;
>+const targetCount = 200;
> var resizeObserver = new ResizeObserver( entries => {
>     for (let entry of entries)
>         entry.target.myEntry = entry;
>@@ -15,10 +15,12 @@ var resizeObserver = new ResizeObserver( entries => {
>     parent.postMessage('Notified', '*');
> });
> 
>+var container = document.createElement('p');
> for (let i = 0; i < targetCount; ++i) {
>     var target = document.createElement('div');
>-    document.body.appendChild(target);
>+    container.appendChild(target);
> }
>+document.body.appendChild(container);
> 
> document.querySelectorAll('div').forEach(target => resizeObserver.observe(target));
>

Actions: View | Formatted Diff | Diff

Attachments on bug 198666: 371636 | 371647 | 371649 | 372449 | 372450