[inspector] Make pausing on console.assert work with multiple sessions

src/inspector/v8-console.cc

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/inspector/v8-console.h"
 
 #include "src/base/macros.h"
 #include "src/inspector/injected-script.h"
 #include "src/inspector/inspected-context.h"
 #include "src/inspector/string-util.h"
@@ skipping 257 matching lines @@
 void V8Console::Assert(const v8::debug::ConsoleCallArguments& info) {
   ConsoleHelper helper(info, m_inspector);
   DCHECK(!helper.firstArgToBoolean(false));
 
   std::vector<v8::Local<v8::Value>> arguments;
   for (int i = 1; i < info.Length(); ++i) arguments.push_back(info[i]);
   if (info.Length() < 2)
     arguments.push_back(
         toV8String(m_inspector->isolate(), String16("console.assert")));
   helper.reportCall(ConsoleAPIType::kAssert, arguments);
-
-  // TODO(dgozman): only break once, not per each session.
-  helper.forEachSession([](V8InspectorSessionImpl* session) {
-    if (session->debuggerAgent()->enabled()) {
-      session->debuggerAgent()->breakProgramOnException(
-          protocol::Debugger::Paused::ReasonEnum::Assert, nullptr);
-    }
-  });
+  m_inspector->debugger()->breakProgramOnAssert(helper.groupId());
 }
 
 void V8Console::MarkTimeline(const v8::debug::ConsoleCallArguments& info) {
   ConsoleHelper(info, m_inspector)
       .reportDeprecatedCall("V8Console#markTimelineDeprecated",
                             "'console.markTimeline' is "
                             "deprecated. Please use "
                             "'console.timeStamp' instead.");
   TimeStamp(info);
 }
@@ skipping 459 matching lines @@
                          ->GetOwnPropertyDescriptor(
                              m_context, v8::Local<v8::String>::Cast(name))
                          .ToLocal(&descriptor);
       DCHECK(success);
       USE(success);
     }
   }
 }
 
 }  // namespace v8_inspector