Bug 84102 - [Mac] fast/profiler/stop-profiling-after-setTimeout.html and fast/profiler/dead-time.html are flaky
Summary: [Mac] fast/profiler/stop-profiling-after-setTimeout.html and fast/profiler/de...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Csaba Osztrogonác
URL:
Keywords: MakingBotsRed
Depends on:
Blocks:
 
Reported: 2012-04-16 17:10 PDT by Anders Carlsson
Modified: 2014-01-27 02:42 PST (History)
7 users (show)

See Also:

Attachments
Patch (6.19 KB, patch)
2014-01-23 07:51 PST, Csaba Osztrogonác 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Carlsson 2012-04-16 17:10:57 PDT 
--- /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/fast/profiler/stop-profiling-after-setTimeout-expected.txt
+++ /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/fast/profiler/stop-profiling-after-setTimeout-actual.txt
@@ -7,6 +7,10 @@
    onload stop-profiling-after-setTimeout.html (line 20)
       startTest stop-profiling-after-setTimeout.html (line 12)
          setTimeout (no file) (line 0)
+   (anonymous function) notification-listeners.html (line 44)
+      isEqual (no file) (line 0)
+   (anonymous function) loading-iframe-sends-notification.html (line 56)
+      indexOf (no file) (line 0)
    (program) stop-profiling-after-setTimeout.html (line 1)
       endTest profiler-test-JS-resources.js (line 1)
 
--- /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/fast/profiler/dead-time-expected.txt
+++ /Volumes/Data/slave/lion-intel-release-tests/build/layout-test-results/fast/profiler/dead-time-actual.txt
@@ -7,6 +7,10 @@
    onload dead-time.html (line 20)
       startTest dead-time.html (line 12)
          setTimeout (no file) (line 0)
+   (anonymous function) notification-listeners.html (line 44)
+      isEqual (no file) (line 0)
+   (anonymous function) loading-iframe-sends-notification.html (line 56)
+      indexOf (no file) (line 0)
    (program) dead-time.html (line 1)
       endTest profiler-test-JS-resources.js (line 1)
Comment 1 Jesus Sanchez-Palencia 2012-05-28 15:22:54 PDT 
Committed r118708: <http://trac.webkit.org/changeset/118708>
Comment 2 Jesus Sanchez-Palencia 2012-05-28 15:25:15 PDT 
(In reply to comment #1)
> Committed r118708: <http://trac.webkit.org/changeset/118708>

Here I've skipped these tests for Qt, EFL and GTK until we can investigate them, after https://bugs.webkit.org/show_bug.cgi?id=42328 . I can see they are already skipped for Mac and Chromium.
Comment 3 Jesus Sanchez-Palencia 2012-05-28 15:26:30 PDT 
Reopening since it was only marked as RESOLVED FIXED by mistake. Sorry for the noise.
Comment 4 Csaba Osztrogonác 2014-01-23 07:51:31 PST 
Created attachment 221980 [details]
Patch

I can't reproduce this flakiness on Mac and EFL now. But the results should be updated and we can remove the unnecessary GTK expected files.
Comment 5 Csaba Osztrogonác 2014-01-27 02:42:03 PST 
Comment on attachment 221980 [details]
Patch

Clearing flags on attachment: 221980

Committed r162832: <http://trac.webkit.org/changeset/162832>
Comment 6 Csaba Osztrogonác 2014-01-27 02:42:09 PST 
All reviewed patches have been landed.  Closing bug.