Avoid using stale InspectedContext pointers

src/inspector/v8-inspector-session-impl.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-inspector-session-impl.h"
 
 #include "src/inspector/injected-script.h"
 #include "src/inspector/inspected-context.h"
 #include "src/inspector/protocol/Protocol.h"
 #include "src/inspector/remote-object-id.h"
@@ skipping 169 matching lines @@
   }
 
   auto contextsIt = contexts->find(contextId);
   if (contextsIt == contexts->end()) {
     *errorString = "Cannot find context with specified id";
     return nullptr;
   }
 
   const std::unique_ptr<InspectedContext>& context = contextsIt->second;
   if (!context->getInjectedScript()) {
-    context->createInjectedScript();
-    if (!context->getInjectedScript()) {
+    if (!context->createInjectedScript()) {
       *errorString = "Cannot access specified execution context";
       return nullptr;
     }
     if (m_customObjectFormatterEnabled)
       context->getInjectedScript()->setCustomObjectFormatterEnabled(true);
   }
   return context->getInjectedScript();
 }
 
 InjectedScript* V8InspectorSessionImpl::findInjectedScript(
@@ skipping 207 matching lines @@
   std::vector<std::unique_ptr<protocol::Debugger::SearchMatch>> matches =
       searchInTextByLinesImpl(this, toString16(text), toString16(query),
                               caseSensitive, isRegex);
   std::vector<std::unique_ptr<protocol::Debugger::API::SearchMatch>> result;
   for (size_t i = 0; i < matches.size(); ++i)
     result.push_back(std::move(matches[i]));
   return result;
 }
 
 }  // namespace v8_inspector