[inspector] align console implementation with spec

src/inspector/inspected-context.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/inspected-context.h"
 
 #include "src/inspector/injected-script.h"
 #include "src/inspector/string-util.h"
 #include "src/inspector/v8-console.h"
 #include "src/inspector/v8-inspector-impl.h"
@@ skipping 30 matching lines @@
       m_humanReadableName(toString16(info.humanReadableName)),
       m_auxData(toString16(info.auxData)),
       m_reported(false) {
   m_context.SetWeak(&m_context, &clearContext,
                     v8::WeakCallbackType::kParameter);
 
   v8::Isolate* isolate = m_inspector->isolate();
   v8::Local<v8::Object> global = info.context->Global();
   v8::Local<v8::Object> console =
       V8Console::createConsole(this, info.hasMemoryOnConsole);
-  if (!global
-           ->Set(info.context, toV8StringInternalized(isolate, "console"),
-                 console)
+  v8::PropertyDescriptor descriptor(console, /* writable */ true);
+  descriptor.set_enumerable(false);
+  descriptor.set_configurable(true);
+  v8::Local<v8::String> consoleKey = toV8StringInternalized(isolate, "console");
+  if (!global->DefineProperty(info.context, consoleKey, descriptor)
            .FromMaybe(false))
     return;
   m_console.Reset(isolate, console);
   m_console.SetWeak();
 }
 
 InspectedContext::~InspectedContext() {
   if (!m_console.IsEmpty()) {
     v8::HandleScope scope(isolate());
     V8Console::clearInspectedContextIfNeeded(context(),
@@ skipping 14 matching lines @@
   std::unique_ptr<InjectedScript> injectedScript = InjectedScript::create(this);
   // InjectedScript::create can destroy |this|.
   if (!injectedScript) return false;
   m_injectedScript = std::move(injectedScript);
   return true;
 }
 
 void InspectedContext::discardInjectedScript() { m_injectedScript.reset(); }
 
 }  // namespace v8_inspector