[DevTools] Make most fields of ExceptionDetails non-optional.

third_party/WebKit/Source/platform/v8_inspector/V8ConsoleMessage.cpp

 // Copyright 2016 The Chromium 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 "platform/v8_inspector/V8ConsoleMessage.h"
 
 #include "platform/v8_inspector/InspectedContext.h"
 #include "platform/v8_inspector/V8ConsoleAgentImpl.h"
 #include "platform/v8_inspector/V8DebuggerImpl.h"
 #include "platform/v8_inspector/V8InspectorSessionImpl.h"
@@ skipping 279 matching lines @@
             args->addItem(std::move(wrapped));
         }
     }
     return args;
 }
 
 void V8ConsoleMessage::reportToFrontend(protocol::Runtime::Frontend* frontend, V8InspectorSessionImpl* session, bool generatePreview) const
 {
     if (m_origin == V8MessageOrigin::kException) {
         // TODO(dgozman): unify with InjectedScript::createExceptionDetails.
-        std::unique_ptr<protocol::Runtime::ExceptionDetails> details = protocol::Runtime::ExceptionDetails::create().setText(m_message).build();
-        details->setUrl(m_url);
-        if (m_lineNumber)
-            details->setLineNumber(static_cast<int>(m_lineNumber) - 1);
-        if (m_columnNumber)
-            details->setColumnNumber(static_cast<int>(m_columnNumber) - 1);
-        if (m_scriptId)
-            details->setScriptId(String16::fromInteger(m_scriptId));
+        std::unique_ptr<protocol::Runtime::ExceptionDetails> details = protocol::Runtime::ExceptionDetails::create()
+            .setText(m_message)
+            .setLineNumber(m_lineNumber ? m_lineNumber - 1 : 0)
+            .setColumnNumber(m_columnNumber ? m_columnNumber - 1 : 0)
+            .setScriptId(m_scriptId ? String16::fromInteger(m_scriptId) : String16())
+            .build();
+        if (!m_url.isEmpty())
+            details->setUrl(m_url);
         if (m_stackTrace)
-            details->setStack(m_stackTrace->buildInspectorObject());
+            details->setStackTrace(m_stackTrace->buildInspectorObject());
 
         std::unique_ptr<protocol::Runtime::RemoteObject> exception = wrapException(session, generatePreview);
 
         if (exception)
             frontend->exceptionThrown(m_exceptionId, m_timestamp, std::move(details), std::move(exception), m_contextId);
         else
             frontend->exceptionThrown(m_exceptionId, m_timestamp, std::move(details));
         return;
     }
     if (m_origin == V8MessageOrigin::kRevokedException) {
@@ skipping 155 matching lines @@
     }
 }
 
 void V8ConsoleMessageStorage::contextDestroyed(int contextId)
 {
     for (size_t i = 0; i < m_messages.size(); ++i)
         m_messages[i]->contextDestroyed(contextId);
 }
 
 } // namespace blink