Changeset 62772 in webkit


Ignore:
Timestamp:
Jul 8, 2010 2:05:30 AM (14 years ago)
Author:
yurys@chromium.org
Message:

2010-07-07 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: skip breakpoints when script is already paused
​https://bugs.webkit.org/show_bug.cgi?id=41768

Test: inspector/debugger-no-nested-pause.html

  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::handleV8DebugEvent):

2010-07-07 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: skip breakpoints when script is already paused
​https://bugs.webkit.org/show_bug.cgi?id=41768

  • inspector/debugger-no-nested-pause-expected.txt: Added.
  • inspector/debugger-no-nested-pause.html: Added.
Location:
trunk
Files:
2 added
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r62769 r62772  
     12010-07-07  Yury Semikhatsky  <yurys@chromium.org>
     2
     3        Reviewed by Pavel Feldman.
     4
     5        Web Inspector: skip breakpoints when script is already paused
     6        https://bugs.webkit.org/show_bug.cgi?id=41768
     7
     8        * inspector/debugger-no-nested-pause-expected.txt: Added.
     9        * inspector/debugger-no-nested-pause.html: Added.
     10
    1112010-07-08  Yury Semikhatsky  <yurys@chromium.org>
    212

  • trunk/LayoutTests/platform/qt/Skipped

    r62657 r62772  
    54395439# [Qt] Regression(r61749): inspector/debugger-pause-on-debugger-statement.html fails
    54405440# https://bugs.webkit.org/show_bug.cgi?id=41147
     5441inspector/debugger-no-nested-pause.html
    54415442inspector/debugger-pause-on-debugger-statement.html
    54425443inspector/debugger-pause-in-eval-script.html

  • trunk/WebCore/ChangeLog

    r62771 r62772  
     12010-07-07  Yury Semikhatsky  <yurys@chromium.org>
     2
     3        Reviewed by Pavel Feldman.
     4
     5        Web Inspector: skip breakpoints when script is already paused
     6        https://bugs.webkit.org/show_bug.cgi?id=41768
     7
     8        Test: inspector/debugger-no-nested-pause.html
     9
     10        * bindings/v8/ScriptDebugServer.cpp:
     11        (WebCore::ScriptDebugServer::handleV8DebugEvent):
     12
    1132010-07-08  Eric Seidel  <eric@webkit.org>
    214

  • trunk/WebCore/bindings/v8/ScriptDebugServer.cpp

    r62754 r62772  
    357357                        return;
    358358                }
     359
     360                // Don't allow nested breaks.
     361                if (m_pausedPage)
     362                    return;
    359363                m_executionState.set(eventDetails.GetExecutionState());
    360364                m_pausedPage = frame->page();
Note: See TracChangeset for help on using the changeset viewer.