<?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>139523</bug_id>
          
          <creation_ts>2014-12-10 18:46:10 -0800</creation_ts>
          <short_desc>Web Inspector: debugger should not require multiple &quot;step over&quot; commands to step past &apos;return&apos; statement</short_desc>
          <delta_ts>2016-09-30 16:42: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>Web Inspector</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>155325</dup_id>
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Brian Burg">burg</reporter>
          <assigned_to name="Joseph Pecoraro">joepeck</assigned_to>
          <cc>bburg</cc>
    
    <cc>graouts</cc>
    
    <cc>joepeck</cc>
    
    <cc>timothy</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1054384</commentid>
    <comment_count>0</comment_count>
    <who name="Brian Burg">burg</who>
    <bug_when>2014-12-10 18:46:10 -0800</bug_when>
    <thetext>Steps to repro:

1. Pause inside a function with |return &lt;expr&gt;;| on the last line.
2. Step over each statement until the return.
3. Step over the return statement.

At the end of step 3, the debugger is stepped &quot;past&quot; the return statement. How this is shown to the user seems inconsistent.
Sometimes, the debugger implicitly &quot;steps out&quot; to the callee&apos;s call frame, while other times it seems to step past the return statement without stepping out.

I haven&apos;t peeked at the protocol traffic to see what commands are actually being emitted. It may be the case that some of the step over clicks were not being sent to the backend somehow. To prevent this sort of race, we could add a debouncing animation to the stepping buttons that makes it clearer when the debugger command has been processed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1056404</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2014-12-17 11:21:26 -0800</bug_when>
    <thetext>&lt;rdar://problem/19281478&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1228848</commentid>
    <comment_count>2</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2016-09-12 12:21:20 -0700</bug_when>
    <thetext>I have a fix for this. I&apos;ll likely duplicate to a more general bug addressing many stepping issues.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1235502</commentid>
    <comment_count>3</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2016-09-30 16:42:30 -0700</bug_when>
    <thetext>Fixed by: bug 155325
&lt;https://trac.webkit.org/changeset/206652&gt;

*** This bug has been marked as a duplicate of bug 155325 ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>