DevTools: Fix disabling all pauses on reload.

Source/core/inspector/InspectorDebuggerAgent.cpp

 /*
  * Copyright (C) 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2010-2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 1007 matching lines @@
     m_frontend->resumed();
 }
 
 bool InspectorDebuggerAgent::canBreakProgram()
 {
     return scriptDebugServer().canBreakProgram();
 }
 
 void InspectorDebuggerAgent::breakProgram(InspectorFrontend::Debugger::Reason::Enum breakReason, PassRefPtr<JSONObject> data)
 {
+    if (m_skipAllPauses)

[yurys, 2013/11/08 06:38:26]

Why shouldSkipBreakpointPause/shouldSkipStepPause are not enough? AFAIU we
shouldn't have come here if those methods work correctly.

[aandrey, 2013/11/08 09:10:39]

The shouldSkip* methods are listener callbacks called from V8 debugger.

The breakProgram() is called from our backend to break the program on
DOM/Event/Xhr breakpoints, console.assert and maybe others, and does break it
w/o checking the shouldSkip* callbacks.

+        return;
     m_breakReason = breakReason;
     m_breakAuxData = data;
     scriptDebugServer().breakProgram();
 }
 
 void InspectorDebuggerAgent::clear()
 {
     m_pausedScriptState = 0;
     m_currentCallStack = ScriptValue();
     m_scripts.clear();
@@ skipping 35 matching lines @@
 void InspectorDebuggerAgent::reset()
 {
     m_scripts.clear();
     m_breakpointIdToDebugServerBreakpointIds.clear();
     if (m_frontend)
         m_frontend->globalObjectCleared();
 }
 
 } // namespace WebCore