DevTools: deoilpanize inspector/v8 and related classes.

third_party/WebKit/Source/core/frame/FrameConsole.cpp

 /*
  * Copyright (C) 2013 Apple 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.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 83 matching lines @@
     } else {
         lineNumber = consoleMessage->lineNumber();
         messageURL = consoleMessage->url();
     }
 
     messageStorage()->reportMessage(m_frame->document(), consoleMessage);
 
     if (consoleMessage->source() == NetworkMessageSource)
         return;
 
-    RefPtrWillBeRawPtr<ScriptCallStack> reportedCallStack = nullptr;
+    RefPtr<ScriptCallStack> reportedCallStack = nullptr;

[sof, 2016/01/20 11:39:50]

(if you like/prefer, there's no need to initialize this RefPtr<> now. here & in
a few other places in this CL.)

[pfeldman, 2016/01/20 17:53:28]

Done.

     if (consoleMessage->source() != ConsoleAPIMessageSource) {
         if (consoleMessage->callStack() && frame().chromeClient().shouldReportDetailedMessageForSource(frame(), messageURL))
             reportedCallStack = consoleMessage->callStack();
     } else {
         if (!frame().host() || (consoleMessage->scriptArguments() && !consoleMessage->scriptArguments()->argumentCount()))
             return;
 
         if (!allClientReportingMessageTypes().contains(consoleMessage->type()))
             return;
 
@@ skipping 14 matching lines @@
     if (response.httpStatusCode() < 400)
         return;
     if (response.wasFallbackRequiredByServiceWorker())
         return;
     String message = "Failed to load resource: the server responded with a status of " + String::number(response.httpStatusCode()) + " (" + response.httpStatusText() + ')';
     RefPtrWillBeRawPtr<ConsoleMessage> consoleMessage = ConsoleMessage::create(NetworkMessageSource, ErrorMessageLevel, message, response.url().string());
     consoleMessage->setRequestIdentifier(requestIdentifier);
     addMessage(consoleMessage.release());
 }
 
-String FrameConsole::formatStackTraceString(const String& originalMessage, PassRefPtrWillBeRawPtr<ScriptCallStack> callStack)
+String FrameConsole::formatStackTraceString(const String& originalMessage, PassRefPtr<ScriptCallStack> callStack)
 {
     StringBuilder stackTrace;
     for (size_t i = 0; i < callStack->size(); ++i) {
         const ScriptCallFrame& frame = callStack->at(i);
         stackTrace.append("\n    at " + (frame.functionName().length() ? frame.functionName() : "(anonymous function)"));
         stackTrace.appendLiteral(" (");
         stackTrace.append(frame.sourceURL());
         stackTrace.append(':');
         stackTrace.appendNumber(frame.lineNumber());
         stackTrace.append(':');
@@ skipping 53 matching lines @@
     consoleMessage->setRequestIdentifier(requestIdentifier);
     storage->reportMessage(m_frame->document(), consoleMessage.release());
 }
 
 DEFINE_TRACE(FrameConsole)
 {
     visitor->trace(m_frame);
 }
 
 } // namespace blink