<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>95414</bug_id>
          
          <creation_ts>2012-08-29 18:27:50 -0700</creation_ts>
          <short_desc>REGRESSION(r127039): It broke inspector/timeline/timeline-load.html fails on many platforms</short_desc>
          <delta_ts>2012-08-30 16:43:30 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>LayoutTestFailure, MakingBotsRed</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>91991</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Roger Fong">roger_fong</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>apavlov</cc>
    
    <cc>ap</cc>
    
    <cc>eustas.bug</cc>
    
    <cc>mikhail.pozdnyakov</cc>
    
    <cc>ossy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>708165</commentid>
    <comment_count>0</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2012-08-29 18:27:50 -0700</bug_when>
    <thetext>inspector/timeline/timeline-load.html:</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708168</commentid>
    <comment_count>1</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2012-08-29 18:30:55 -0700</bug_when>
    <thetext>Test fails on Apple-Windows port :( any idea why?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708179</commentid>
    <comment_count>2</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2012-08-29 18:38:37 -0700</bug_when>
    <thetext>diff: 

--- /home/buildbot/slave/win-release-tests/build/layout-test-results/inspector/timeline/timeline-load-expected.txt	2012-08-29 13:57:46.075447000 -0700
+++ /home/buildbot/slave/win-release-tests/build/layout-test-results/inspector/timeline/timeline-load-actual.txt	2012-08-29 13:57:46.074446900 -0700
@@ -9,6 +9,3 @@
 Running: testIncompatible
 passed
 
-Running: testMalformed
-passed
-</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708229</commentid>
    <comment_count>3</comment_count>
    <who name="">eustas.bug</who>
    <bug_when>2012-08-29 19:44:51 -0700</bug_when>
    <thetext>Last test case checks try-catch execution branch in TimelineModel.js: WebInspector.TimelineModelLoader.transferChunk

Try to add &quot;window.console.log(&apos;something&apos;);&quot; around this code to see if it is the source of the problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708383</commentid>
    <comment_count>4</comment_count>
    <who name="Mikhail Pozdnyakov">mikhail.pozdnyakov</who>
    <bug_when>2012-08-30 00:51:12 -0700</bug_when>
    <thetext>Have same problem on EFL port</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708523</commentid>
    <comment_count>5</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-08-30 05:35:22 -0700</bug_when>
    <thetext>It isn&apos;t a win specific bug, it is a regression caused by r127039
and affedted platforms are: Win, EFL, Qt Debug

here is the Qt debug diff too:
--- /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/layout-test-results/inspector/timeline/timeline-load-expected.txt 
+++ /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/layout-test-results/inspector/timeline/timeline-load-actual.txt 
@@ -6,9 +6,4 @@
 Running: testNormal
 passed
 
-Running: testIncompatible
-passed
 
-Running: testMalformed
-passed
-</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>709024</commentid>
    <comment_count>6</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2012-08-30 14:14:34 -0700</bug_when>
    <thetext>the transferChunk code does not get called. In fact, from what I can tell, the last test isn&apos;t being run at all...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>709073</commentid>
    <comment_count>7</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2012-08-30 15:05:29 -0700</bug_when>
    <thetext>Also would like to note that it seems like the 3rd test in the test suite just isn&apos;t being run in the Windows case.

If I switch the orders of the test cases, (malformed first, incompatible second), malformed test passes fine, incompatible test does not get run.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>709139</commentid>
    <comment_count>8</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2012-08-30 16:34:04 -0700</bug_when>
    <thetext>Upon further investigation I&apos;m seeing strange behavior in InspectorTest.completeTest.

In the first test, the code path makes it through the completeTest call and moves on to test the second test. The second test does not make it through completeTest and the third test does not get called.

I can put the tests in any order and only the first 2 will ever be run.

I would try to debug further...
However, depending on where and what I log, the results change, which is madness.
(Logging integers vs string literals changes things unexpectedly).

I&apos;m adding this test to the Windows skip list for now to get the bots greener.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>709149</commentid>
    <comment_count>9</comment_count>
    <who name="Roger Fong">roger_fong</who>
    <bug_when>2012-08-30 16:43:30 -0700</bug_when>
    <thetext>It was fixed here this morning apparently...
http://trac.webkit.org/changeset/127135
by removing the completeTest call...

Sigh, closing bug...</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>