<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[162264] trunk/Source</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/162264">162264</a></dd>
<dt>Author</dt> <dd>bburg@apple.com</dd>
<dt>Date</dt> <dd>2014-01-18 11:15:33 -0800 (Sat, 18 Jan 2014)</dd>
</dl>
<h3>Log Message</h3>
<pre>Web Inspector: Page should use std::unique_ptr for InspectorController
https://bugs.webkit.org/show_bug.cgi?id=127068
Reviewed by Joseph Pecoraro.
Source/WebCore:
Make Page and WorkerGlobalScope keep a std::unique_ptr to InspectorController
and WorkerInspectorController. Store page references and return controller
references where possible.
Convert call sites to use an InspectorController reference.
Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
Convert InspectorOverlay to keep Page references.
* WebCore.exp.in:
* bindings/js/JSDOMWindowBase.cpp:
(WebCore::JSDOMWindowBase::supportsProfiling):
(WebCore::JSDOMWindowBase::supportsRichSourceInfo):
* dom/Node.cpp:
(WebCore::Node::inspect):
* inspector/InspectorController.cpp:
(WebCore::InspectorController::InspectorController):
(WebCore::InspectorController::inspectedPageDestroyed):
(WebCore::InspectorController::setInspectorFrontendClient):
(WebCore::InspectorController::hasInspectorFrontendClient):
(WebCore::InspectorController::connectFrontend):
(WebCore::InspectorController::disconnectFrontend):
(WebCore::InspectorController::inspectedPage):
(WebCore::InspectorController::developerExtrasEnabled):
* inspector/InspectorController.h:
* inspector/InspectorFrontendClientLocal.cpp:
(WebCore::InspectorFrontendClientLocal::canAttachWindow):
(WebCore::InspectorFrontendClientLocal::changeAttachedWindowHeight):
(WebCore::InspectorFrontendClientLocal::changeAttachedWindowWidth):
(WebCore::InspectorFrontendClientLocal::openInNewTab):
(WebCore::InspectorFrontendClientLocal::restoreAttachedWindowHeight):
* inspector/InspectorOverlay.cpp:
(WebCore::InspectorOverlay::InspectorOverlay):
(WebCore::InspectorOverlay::getHighlight):
(WebCore::InspectorOverlay::highlightQuad):
(WebCore::InspectorOverlay::update):
(WebCore::InspectorOverlay::buildObjectForHighlightedNode):
(WebCore::InspectorOverlay::drawQuadHighlight):
(WebCore::InspectorOverlay::overlayPage):
(WebCore::InspectorOverlay::reset):
* inspector/InspectorOverlay.h:
* inspector/InstrumentingAgents.cpp:
(WebCore::instrumentationForPage): remove null checks.
(WebCore::instrumentationForWorkerGlobalScope): remove null checks.
* inspector/WorkerInspectorController.cpp:
(WebCore::WorkerInspectorController::WorkerInspectorController):
(WebCore::WorkerInspectorController::connectFrontend):
(WebCore::WorkerInspectorController::disconnectFrontend):
* inspector/WorkerInspectorController.h:
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::continueLoadAfterNavigationPolicy):
(WebCore::FrameLoader::dispatchDidClearWindowObjectInWorld):
* page/ContextMenuController.cpp:
(WebCore::ContextMenuController::showContextMenu):
(WebCore::ContextMenuController::contextMenuItemSelected):
(WebCore::ContextMenuController::populate):
(WebCore::ContextMenuController::addInspectElementItem):
* page/FrameView.cpp:
(WebCore::FrameView::sendResizeEventIfNeeded):
* page/Page.cpp:
(WebCore::Page::Page):
* page/Page.h:
(WebCore::Page::inspectorController):
* page/PageDebuggable.cpp:
(WebCore::PageDebuggable::hasLocalDebugger):
(WebCore::PageDebuggable::parentProcessIdentifier):
(WebCore::PageDebuggable::connect):
(WebCore::PageDebuggable::disconnect):
(WebCore::PageDebuggable::dispatchMessageFromRemoteFrontend):
(WebCore::PageDebuggable::setIndicating):
* platform/graphics/texmap/coordinated/CompositingCoordinator.cpp:
(WebCore::CompositingCoordinator::syncDisplayState):
* testing/Internals.cpp:
(WebCore::Internals::resetToConsistentState):
(WebCore::Internals::inspectorHighlightRects):
(WebCore::Internals::inspectorHighlightObject):
(WebCore::Internals::emitInspectorDidBeginFrame):
(WebCore::Internals::emitInspectorDidCancelFrame):
(WebCore::Internals::openDummyInspectorFrontend):
(WebCore::Internals::closeDummyInspectorFrontend):
(WebCore::Internals::setInspectorResourcesDataSizeLimits):
(WebCore::Internals::setJavaScriptProfilingEnabled):
* workers/WorkerGlobalScope.cpp:
(WebCore::WorkerGlobalScope::WorkerGlobalScope):
* workers/WorkerGlobalScope.h: remove clearInspector().
(WebCore::WorkerGlobalScope::workerInspectorController):
* workers/WorkerMessagingProxy.cpp:
(WebCore::connectToWorkerGlobalScopeInspectorTask):
(WebCore::disconnectFromWorkerGlobalScopeInspectorTask):
(WebCore::dispatchOnInspectorBackendTask):
* workers/WorkerThread.cpp:
(WebCore::WorkerThreadShutdownFinishTask::performTask):
Source/WebKit/efl:
Convert call sites to use a InspectorController reference.
Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
* WebCoreSupport/DumpRenderTreeSupportEfl.cpp:
(DumpRenderTreeSupportEfl::evaluateInWebInspector):
* WebCoreSupport/InspectorClientEfl.cpp:
(WebCore::InspectorClientEfl::openInspectorFrontend):
* ewk/ewk_view.cpp:
(ewk_view_inspector_show):
(ewk_view_inspector_close):
Source/WebKit/gtk:
Convert call sites to use a InspectorController reference.
Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
* WebCoreSupport/ChromeClientGtk.cpp:
(WebKit::paintWebView):
* WebCoreSupport/InspectorClientGtk.cpp:
(WebKit::InspectorClient::openInspectorFrontend):
* webkit/webkitwebinspector.cpp:
(webkit_web_inspector_set_property):
(webkit_web_inspector_get_property):
(webkit_web_inspector_show):
(webkit_web_inspector_inspect_node):
(webkit_web_inspector_inspect_coordinates):
(webkit_web_inspector_close):
(webkit_web_inspector_execute_script):
Source/WebKit/mac:
Convert call sites to use a InspectorController reference.
Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
* WebCoreSupport/WebInspectorClient.mm:
(WebInspectorClient::openInspectorFrontend):
(-[WebInspectorWindowController destroyInspectorView:]):
* WebInspector/WebInspector.mm:
(-[WebInspector showWindow]):
(-[WebInspector isJavaScriptProfilingEnabled]):
(-[WebInspector setJavaScriptProfilingEnabled:]):
(-[WebInspector close:]):
(-[WebInspector evaluateInFrontend:callId:script:]):
* WebInspector/WebNodeHighlighter.mm:
(-[WebNodeHighlighter highlight]):
Source/WebKit/win:
Convert call sites to use a InspectorController reference.
* WebCoreSupport/WebInspectorClient.cpp:
(WebInspectorClient::openInspectorFrontend):
(WebInspectorFrontendClient::destroyInspectorView):
(WebInspectorFrontendClient::onClose):
* WebInspector.cpp:
(WebInspector::show):
(WebInspector::close):
(WebInspector::isJavaScriptProfilingEnabled):
(WebInspector::setJavaScriptProfilingEnabled):
(WebInspector::evaluateInFrontend):
* WebNodeHighlight.cpp:
(WebNodeHighlight::update):
Source/WebKit2:
Convert call sites to use a InspectorController reference.
* WebProcess/WebCoreSupport/WebInspectorClient.cpp:
(WebKit::WebInspectorClient::drawRect):
* WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp:
(WebKit::WebInspectorFrontendClient::WebInspectorFrontendClient):
(WebKit::WebInspectorFrontendClient::closeWindow):
* WebProcess/WebPage/WebInspector.cpp:
(WebKit::WebInspector::createInspectorPage):
(WebKit::WebInspector::show):
(WebKit::WebInspector::close):
(WebKit::WebInspector::evaluateScriptForTest):
(WebKit::WebInspector::showConsole):
(WebKit::WebInspector::showResources):
(WebKit::WebInspector::showMainResourceForFrame):
(WebKit::WebInspector::startJavaScriptDebugging):
(WebKit::WebInspector::stopJavaScriptDebugging):
(WebKit::WebInspector::setJavaScriptProfilingEnabled):
(WebKit::WebInspector::startJavaScriptProfiling):
(WebKit::WebInspector::stopJavaScriptProfiling):
(WebKit::WebInspector::startPageProfiling):
(WebKit::WebInspector::stopPageProfiling):
(WebKit::WebInspector::dispatchMessageFromRemoteFrontend):
(WebKit::WebInspector::remoteFrontendConnected):
(WebKit::WebInspector::remoteFrontendDisconnected):</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreWebCoreexpin">trunk/Source/WebCore/WebCore.exp.in</a></li>
<li><a href="#trunkSourceWebCorebindingsjsJSDOMWindowBasecpp">trunk/Source/WebCore/bindings/js/JSDOMWindowBase.cpp</a></li>
<li><a href="#trunkSourceWebCoredomNodecpp">trunk/Source/WebCore/dom/Node.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorControllercpp">trunk/Source/WebCore/inspector/InspectorController.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorControllerh">trunk/Source/WebCore/inspector/InspectorController.h</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorFrontendClientLocalcpp">trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorOverlaycpp">trunk/Source/WebCore/inspector/InspectorOverlay.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorOverlayh">trunk/Source/WebCore/inspector/InspectorOverlay.h</a></li>
<li><a href="#trunkSourceWebCoreinspectorInstrumentingAgentscpp">trunk/Source/WebCore/inspector/InstrumentingAgents.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorWorkerInspectorControllercpp">trunk/Source/WebCore/inspector/WorkerInspectorController.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorWorkerInspectorControllerh">trunk/Source/WebCore/inspector/WorkerInspectorController.h</a></li>
<li><a href="#trunkSourceWebCoreloaderFrameLoadercpp">trunk/Source/WebCore/loader/FrameLoader.cpp</a></li>
<li><a href="#trunkSourceWebCorepageContextMenuControllercpp">trunk/Source/WebCore/page/ContextMenuController.cpp</a></li>
<li><a href="#trunkSourceWebCorepageFrameViewcpp">trunk/Source/WebCore/page/FrameView.cpp</a></li>
<li><a href="#trunkSourceWebCorepagePagecpp">trunk/Source/WebCore/page/Page.cpp</a></li>
<li><a href="#trunkSourceWebCorepagePageh">trunk/Source/WebCore/page/Page.h</a></li>
<li><a href="#trunkSourceWebCorepagePageDebuggablecpp">trunk/Source/WebCore/page/PageDebuggable.cpp</a></li>
<li><a href="#trunkSourceWebCoreplatformgraphicstexmapcoordinatedCompositingCoordinatorcpp">trunk/Source/WebCore/platform/graphics/texmap/coordinated/CompositingCoordinator.cpp</a></li>
<li><a href="#trunkSourceWebCoretestingInternalscpp">trunk/Source/WebCore/testing/Internals.cpp</a></li>
<li><a href="#trunkSourceWebCoreworkersWorkerGlobalScopecpp">trunk/Source/WebCore/workers/WorkerGlobalScope.cpp</a></li>
<li><a href="#trunkSourceWebCoreworkersWorkerGlobalScopeh">trunk/Source/WebCore/workers/WorkerGlobalScope.h</a></li>
<li><a href="#trunkSourceWebCoreworkersWorkerMessagingProxycpp">trunk/Source/WebCore/workers/WorkerMessagingProxy.cpp</a></li>
<li><a href="#trunkSourceWebCoreworkersWorkerThreadcpp">trunk/Source/WebCore/workers/WorkerThread.cpp</a></li>
<li><a href="#trunkSourceWebKiteflChangeLog">trunk/Source/WebKit/efl/ChangeLog</a></li>
<li><a href="#trunkSourceWebKiteflWebCoreSupportDumpRenderTreeSupportEflcpp">trunk/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp</a></li>
<li><a href="#trunkSourceWebKiteflWebCoreSupportInspectorClientEflcpp">trunk/Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.cpp</a></li>
<li><a href="#trunkSourceWebKiteflewkewk_viewcpp">trunk/Source/WebKit/efl/ewk/ewk_view.cpp</a></li>
<li><a href="#trunkSourceWebKitgtkChangeLog">trunk/Source/WebKit/gtk/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitgtkWebCoreSupportChromeClientGtkcpp">trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp</a></li>
<li><a href="#trunkSourceWebKitgtkWebCoreSupportInspectorClientGtkcpp">trunk/Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.cpp</a></li>
<li><a href="#trunkSourceWebKitgtkwebkitwebkitwebinspectorcpp">trunk/Source/WebKit/gtk/webkit/webkitwebinspector.cpp</a></li>
<li><a href="#trunkSourceWebKitmacChangeLog">trunk/Source/WebKit/mac/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitmacWebCoreSupportWebInspectorClientmm">trunk/Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm</a></li>
<li><a href="#trunkSourceWebKitmacWebInspectorWebInspectormm">trunk/Source/WebKit/mac/WebInspector/WebInspector.mm</a></li>
<li><a href="#trunkSourceWebKitmacWebInspectorWebNodeHighlightermm">trunk/Source/WebKit/mac/WebInspector/WebNodeHighlighter.mm</a></li>
<li><a href="#trunkSourceWebKitwinChangeLog">trunk/Source/WebKit/win/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitwinWebCoreSupportWebInspectorClientcpp">trunk/Source/WebKit/win/WebCoreSupport/WebInspectorClient.cpp</a></li>
<li><a href="#trunkSourceWebKitwinWebInspectorcpp">trunk/Source/WebKit/win/WebInspector.cpp</a></li>
<li><a href="#trunkSourceWebKitwinWebNodeHighlightcpp">trunk/Source/WebKit/win/WebNodeHighlight.cpp</a></li>
<li><a href="#trunkSourceWebKit2ChangeLog">trunk/Source/WebKit2/ChangeLog</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebCoreSupportWebInspectorClientcpp">trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorClient.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebCoreSupportWebInspectorFrontendClientcpp">trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebPageWebInspectorcpp">trunk/Source/WebKit2/WebProcess/WebPage/WebInspector.cpp</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/ChangeLog        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -1,3 +1,102 @@
</span><ins>+2014-01-18 Brian Burg <bburg@apple.com>
+
+ Web Inspector: Page should use std::unique_ptr for InspectorController
+ https://bugs.webkit.org/show_bug.cgi?id=127068
+
+ Reviewed by Joseph Pecoraro.
+
+ Make Page and WorkerGlobalScope keep a std::unique_ptr to InspectorController
+ and WorkerInspectorController. Store page references and return controller
+ references where possible.
+
+ Convert call sites to use an InspectorController reference.
+ Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
+ Convert InspectorOverlay to keep Page references.
+
+ * WebCore.exp.in:
+ * bindings/js/JSDOMWindowBase.cpp:
+ (WebCore::JSDOMWindowBase::supportsProfiling):
+ (WebCore::JSDOMWindowBase::supportsRichSourceInfo):
+ * dom/Node.cpp:
+ (WebCore::Node::inspect):
+ * inspector/InspectorController.cpp:
+ (WebCore::InspectorController::InspectorController):
+ (WebCore::InspectorController::inspectedPageDestroyed):
+ (WebCore::InspectorController::setInspectorFrontendClient):
+ (WebCore::InspectorController::hasInspectorFrontendClient):
+ (WebCore::InspectorController::connectFrontend):
+ (WebCore::InspectorController::disconnectFrontend):
+ (WebCore::InspectorController::inspectedPage):
+ (WebCore::InspectorController::developerExtrasEnabled):
+ * inspector/InspectorController.h:
+ * inspector/InspectorFrontendClientLocal.cpp:
+ (WebCore::InspectorFrontendClientLocal::canAttachWindow):
+ (WebCore::InspectorFrontendClientLocal::changeAttachedWindowHeight):
+ (WebCore::InspectorFrontendClientLocal::changeAttachedWindowWidth):
+ (WebCore::InspectorFrontendClientLocal::openInNewTab):
+ (WebCore::InspectorFrontendClientLocal::restoreAttachedWindowHeight):
+ * inspector/InspectorOverlay.cpp:
+ (WebCore::InspectorOverlay::InspectorOverlay):
+ (WebCore::InspectorOverlay::getHighlight):
+ (WebCore::InspectorOverlay::highlightQuad):
+ (WebCore::InspectorOverlay::update):
+ (WebCore::InspectorOverlay::buildObjectForHighlightedNode):
+ (WebCore::InspectorOverlay::drawQuadHighlight):
+ (WebCore::InspectorOverlay::overlayPage):
+ (WebCore::InspectorOverlay::reset):
+ * inspector/InspectorOverlay.h:
+ * inspector/InstrumentingAgents.cpp:
+ (WebCore::instrumentationForPage): remove null checks.
+ (WebCore::instrumentationForWorkerGlobalScope): remove null checks.
+ * inspector/WorkerInspectorController.cpp:
+ (WebCore::WorkerInspectorController::WorkerInspectorController):
+ (WebCore::WorkerInspectorController::connectFrontend):
+ (WebCore::WorkerInspectorController::disconnectFrontend):
+ * inspector/WorkerInspectorController.h:
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::continueLoadAfterNavigationPolicy):
+ (WebCore::FrameLoader::dispatchDidClearWindowObjectInWorld):
+ * page/ContextMenuController.cpp:
+ (WebCore::ContextMenuController::showContextMenu):
+ (WebCore::ContextMenuController::contextMenuItemSelected):
+ (WebCore::ContextMenuController::populate):
+ (WebCore::ContextMenuController::addInspectElementItem):
+ * page/FrameView.cpp:
+ (WebCore::FrameView::sendResizeEventIfNeeded):
+ * page/Page.cpp:
+ (WebCore::Page::Page):
+ * page/Page.h:
+ (WebCore::Page::inspectorController):
+ * page/PageDebuggable.cpp:
+ (WebCore::PageDebuggable::hasLocalDebugger):
+ (WebCore::PageDebuggable::parentProcessIdentifier):
+ (WebCore::PageDebuggable::connect):
+ (WebCore::PageDebuggable::disconnect):
+ (WebCore::PageDebuggable::dispatchMessageFromRemoteFrontend):
+ (WebCore::PageDebuggable::setIndicating):
+ * platform/graphics/texmap/coordinated/CompositingCoordinator.cpp:
+ (WebCore::CompositingCoordinator::syncDisplayState):
+ * testing/Internals.cpp:
+ (WebCore::Internals::resetToConsistentState):
+ (WebCore::Internals::inspectorHighlightRects):
+ (WebCore::Internals::inspectorHighlightObject):
+ (WebCore::Internals::emitInspectorDidBeginFrame):
+ (WebCore::Internals::emitInspectorDidCancelFrame):
+ (WebCore::Internals::openDummyInspectorFrontend):
+ (WebCore::Internals::closeDummyInspectorFrontend):
+ (WebCore::Internals::setInspectorResourcesDataSizeLimits):
+ (WebCore::Internals::setJavaScriptProfilingEnabled):
+ * workers/WorkerGlobalScope.cpp:
+ (WebCore::WorkerGlobalScope::WorkerGlobalScope):
+ * workers/WorkerGlobalScope.h: remove clearInspector().
+ (WebCore::WorkerGlobalScope::workerInspectorController):
+ * workers/WorkerMessagingProxy.cpp:
+ (WebCore::connectToWorkerGlobalScopeInspectorTask):
+ (WebCore::disconnectFromWorkerGlobalScopeInspectorTask):
+ (WebCore::dispatchOnInspectorBackendTask):
+ * workers/WorkerThread.cpp:
+ (WebCore::WorkerThreadShutdownFinishTask::performTask):
+
</ins><span class="cx"> 2014-01-18 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><span class="cx"> Remove ENABLE_THREADED_HTML_PARSER defines everywhere
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCoreexpin"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.exp.in (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.exp.in        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/WebCore.exp.in        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -2764,7 +2764,7 @@
</span><span class="cx"> __ZN7WebCore19InspectorController18disconnectFrontendEv
</span><span class="cx"> __ZN7WebCore19InspectorController18setProfilerEnabledEb
</span><span class="cx"> __ZN7WebCore19InspectorController25evaluateForTestInFrontendElRKN3WTF6StringE
</span><del>-__ZN7WebCore19InspectorController26setInspectorFrontendClientEN3WTF10PassOwnPtrINS_23InspectorFrontendClientEEE
</del><ins>+__ZN7WebCore19InspectorController26setInspectorFrontendClientENSt3__110unique_ptrINS_23InspectorFrontendClientENS1_14default_deleteIS3_EEEE
</ins><span class="cx"> __ZN7WebCore19InspectorController39setResourcesDataSizeLimitsFromInternalsEii
</span><span class="cx"> __ZN7WebCore19InspectorController4showEv
</span><span class="cx"> __ZN7WebCore19InspectorController5closeEv
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsjsJSDOMWindowBasecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/js/JSDOMWindowBase.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/js/JSDOMWindowBase.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/bindings/js/JSDOMWindowBase.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -117,7 +117,7 @@
</span><span class="cx"> return false;
</span><span class="cx">
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- return page->inspectorController()->profilerEnabled();
</del><ins>+ return page->inspectorController().profilerEnabled();
</ins><span class="cx"> #else
</span><span class="cx"> return false;
</span><span class="cx"> #endif // ENABLE(INSPECTOR)
</span><span class="lines">@@ -139,7 +139,7 @@
</span><span class="cx"> if (!page)
</span><span class="cx"> return false;
</span><span class="cx">
</span><del>- bool enabled = page->inspectorController()->enabled();
</del><ins>+ bool enabled = page->inspectorController().enabled();
</ins><span class="cx"> ASSERT(enabled || !thisObject->debugger());
</span><span class="cx"> ASSERT(enabled || !supportsProfiling(thisObject));
</span><span class="cx"> return enabled;
</span></span></pre></div>
<a id="trunkSourceWebCoredomNodecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/dom/Node.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/dom/Node.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/dom/Node.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -557,7 +557,7 @@
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><span class="cx"> if (document().page())
</span><del>- document().page()->inspectorController()->inspect(this);
</del><ins>+ document().page()->inspectorController().inspect(this);
</ins><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorController.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorController.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/InspectorController.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -82,10 +82,10 @@
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-InspectorController::InspectorController(Page* page, InspectorClient* inspectorClient)
</del><ins>+InspectorController::InspectorController(Page& page, InspectorClient* inspectorClient)
</ins><span class="cx"> : m_instrumentingAgents(InstrumentingAgents::create(*this))
</span><span class="cx"> , m_injectedScriptManager(std::make_unique<PageInjectedScriptManager>(*this, PageInjectedScriptHost::create()))
</span><del>- , m_overlay(InspectorOverlay::create(page, inspectorClient))
</del><ins>+ , m_overlay(std::make_unique<InspectorOverlay>(page, inspectorClient))
</ins><span class="cx"> , m_inspectorFrontendChannel(nullptr)
</span><span class="cx"> , m_page(page)
</span><span class="cx"> , m_inspectorClient(inspectorClient)
</span><span class="lines">@@ -101,7 +101,7 @@
</span><span class="cx"> m_instrumentingAgents->setInspectorAgent(m_inspectorAgent);
</span><span class="cx"> m_agents.append(std::move(inspectorAgentPtr));
</span><span class="cx">
</span><del>- auto pageAgentPtr = std::make_unique<InspectorPageAgent>(m_instrumentingAgents.get(), page, inspectorClient, m_overlay.get());
</del><ins>+ auto pageAgentPtr = std::make_unique<InspectorPageAgent>(m_instrumentingAgents.get(), &page, inspectorClient, m_overlay.get());
</ins><span class="cx"> InspectorPageAgent* pageAgent = pageAgentPtr.get();
</span><span class="cx"> m_pageAgent = pageAgentPtr.get();
</span><span class="cx"> m_agents.append(std::move(pageAgentPtr));
</span><span class="lines">@@ -137,7 +137,7 @@
</span><span class="cx"> m_resourceAgent = resourceAgentPtr.get();
</span><span class="cx"> m_agents.append(std::move(resourceAgentPtr));
</span><span class="cx">
</span><del>- auto runtimeAgentPtr = std::make_unique<PageRuntimeAgent>(m_instrumentingAgents.get(), m_injectedScriptManager.get(), page, pageAgent);
</del><ins>+ auto runtimeAgentPtr = std::make_unique<PageRuntimeAgent>(m_instrumentingAgents.get(), m_injectedScriptManager.get(), &page, pageAgent);
</ins><span class="cx"> InspectorRuntimeAgent* runtimeAgent = runtimeAgentPtr.get();
</span><span class="cx"> m_agents.append(std::move(runtimeAgentPtr));
</span><span class="cx">
</span><span class="lines">@@ -154,7 +154,7 @@
</span><span class="cx"> m_domDebuggerAgent = domDebuggerAgentPtr.get();
</span><span class="cx"> m_agents.append(std::move(domDebuggerAgentPtr));
</span><span class="cx">
</span><del>- auto profilerAgentPtr = InspectorProfilerAgent::create(m_instrumentingAgents.get(), consoleAgent, page, m_injectedScriptManager.get());
</del><ins>+ auto profilerAgentPtr = InspectorProfilerAgent::create(m_instrumentingAgents.get(), consoleAgent, &page, m_injectedScriptManager.get());
</ins><span class="cx"> m_profilerAgent = profilerAgentPtr.get();
</span><span class="cx"> m_agents.append(std::move(profilerAgentPtr));
</span><span class="cx">
</span><span class="lines">@@ -165,7 +165,7 @@
</span><span class="cx">
</span><span class="cx"> m_agents.append(std::make_unique<InspectorCanvasAgent>(m_instrumentingAgents.get(), pageAgent, m_injectedScriptManager.get()));
</span><span class="cx">
</span><del>- m_agents.append(std::make_unique<InspectorInputAgent>(m_instrumentingAgents.get(), page));
</del><ins>+ m_agents.append(std::make_unique<InspectorInputAgent>(m_instrumentingAgents.get(), &page));
</ins><span class="cx">
</span><span class="cx"> #if USE(ACCELERATED_COMPOSITING)
</span><span class="cx"> m_agents.append(std::make_unique<InspectorLayerTreeAgent>(m_instrumentingAgents.get()));
</span><span class="lines">@@ -195,23 +195,17 @@
</span><span class="cx"> ASSERT(!m_inspectorClient);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-PassOwnPtr<InspectorController> InspectorController::create(Page* page, InspectorClient* client)
-{
- return adoptPtr(new InspectorController(page, client));
-}
-
</del><span class="cx"> void InspectorController::inspectedPageDestroyed()
</span><span class="cx"> {
</span><span class="cx"> disconnectFrontend();
</span><span class="cx"> m_injectedScriptManager->disconnect();
</span><span class="cx"> m_inspectorClient->inspectorDestroyed();
</span><span class="cx"> m_inspectorClient = 0;
</span><del>- m_page = 0;
</del><span class="cx"> }
</span><span class="cx">
</span><del>-void InspectorController::setInspectorFrontendClient(PassOwnPtr<InspectorFrontendClient> inspectorFrontendClient)
</del><ins>+void InspectorController::setInspectorFrontendClient(std::unique_ptr<InspectorFrontendClient> inspectorFrontendClient)
</ins><span class="cx"> {
</span><del>- m_inspectorFrontendClient = inspectorFrontendClient;
</del><ins>+ m_inspectorFrontendClient = std::move(inspectorFrontendClient);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool InspectorController::hasLocalFrontend() const
</span><span class="lines">@@ -234,7 +228,7 @@
</span><span class="cx">
</span><span class="cx"> bool InspectorController::hasInspectorFrontendClient() const
</span><span class="cx"> {
</span><del>- return m_inspectorFrontendClient;
</del><ins>+ return m_inspectorFrontendClient.get();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void InspectorController::didClearWindowObjectInWorld(Frame* frame, DOMWrapperWorld& world)
</span><span class="lines">@@ -268,7 +262,7 @@
</span><span class="cx">
</span><span class="cx"> #if ENABLE(REMOTE_INSPECTOR)
</span><span class="cx"> if (!m_hasRemoteFrontend)
</span><del>- m_page->remoteInspectorInformationDidChange();
</del><ins>+ m_page.remoteInspectorInformationDidChange();
</ins><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -290,7 +284,7 @@
</span><span class="cx">
</span><span class="cx"> #if ENABLE(REMOTE_INSPECTOR)
</span><span class="cx"> if (!m_hasRemoteFrontend)
</span><del>- m_page->remoteInspectorInformationDidChange();
</del><ins>+ m_page.remoteInspectorInformationDidChange();
</ins><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -365,7 +359,7 @@
</span><span class="cx"> return developerExtrasEnabled();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-Page* InspectorController::inspectedPage() const
</del><ins>+Page& InspectorController::inspectedPage() const
</ins><span class="cx"> {
</span><span class="cx"> return m_page;
</span><span class="cx"> }
</span><span class="lines">@@ -453,10 +447,7 @@
</span><span class="cx">
</span><span class="cx"> bool InspectorController::developerExtrasEnabled() const
</span><span class="cx"> {
</span><del>- if (!m_page)
- return false;
-
- return m_page->settings().developerExtrasEnabled();
</del><ins>+ return m_page.settings().developerExtrasEnabled();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool InspectorController::canAccessInspectedScriptState(JSC::ExecState* scriptState) const
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorController.h (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorController.h        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/InspectorController.h        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -77,18 +77,18 @@
</span><span class="cx"> WTF_MAKE_NONCOPYABLE(InspectorController);
</span><span class="cx"> WTF_MAKE_FAST_ALLOCATED;
</span><span class="cx"> public:
</span><ins>+ InspectorController(Page&, InspectorClient*);
</ins><span class="cx"> virtual ~InspectorController();
</span><span class="cx">
</span><del>- static PassOwnPtr<InspectorController> create(Page*, InspectorClient*);
</del><span class="cx"> void inspectedPageDestroyed();
</span><span class="cx">
</span><span class="cx"> bool enabled() const;
</span><del>- Page* inspectedPage() const;
</del><ins>+ Page& inspectedPage() const;
</ins><span class="cx">
</span><span class="cx"> void show();
</span><span class="cx"> void close();
</span><span class="cx">
</span><del>- void setInspectorFrontendClient(PassOwnPtr<InspectorFrontendClient>);
</del><ins>+ void setInspectorFrontendClient(std::unique_ptr<InspectorFrontendClient>);
</ins><span class="cx"> bool hasInspectorFrontendClient() const;
</span><span class="cx"> void didClearWindowObjectInWorld(Frame*, DOMWrapperWorld&);
</span><span class="cx">
</span><span class="lines">@@ -144,13 +144,11 @@
</span><span class="cx"> virtual void didCallInjectedScriptFunction() override;
</span><span class="cx">
</span><span class="cx"> private:
</span><del>- InspectorController(Page*, InspectorClient*);
-
</del><span class="cx"> friend InstrumentingAgents* instrumentationForPage(Page*);
</span><span class="cx">
</span><span class="cx"> RefPtr<InstrumentingAgents> m_instrumentingAgents;
</span><span class="cx"> std::unique_ptr<PageInjectedScriptManager> m_injectedScriptManager;
</span><del>- OwnPtr<InspectorOverlay> m_overlay;
</del><ins>+ std::unique_ptr<InspectorOverlay> m_overlay;
</ins><span class="cx">
</span><span class="cx"> Inspector::InspectorAgent* m_inspectorAgent;
</span><span class="cx"> InspectorDOMAgent* m_domAgent;
</span><span class="lines">@@ -164,9 +162,9 @@
</span><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> RefPtr<Inspector::InspectorBackendDispatcher> m_inspectorBackendDispatcher;
</span><del>- OwnPtr<InspectorFrontendClient> m_inspectorFrontendClient;
</del><ins>+ std::unique_ptr<InspectorFrontendClient> m_inspectorFrontendClient;
</ins><span class="cx"> Inspector::InspectorFrontendChannel* m_inspectorFrontendChannel;
</span><del>- Page* m_page;
</del><ins>+ Page& m_page;
</ins><span class="cx"> InspectorClient* m_inspectorClient;
</span><span class="cx"> Inspector::InspectorAgentRegistry m_agents;
</span><span class="cx"> Vector<InspectorInstrumentationCookie, 2> m_injectedScriptInstrumentationCookies;
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorFrontendClientLocalcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -182,8 +182,8 @@
</span><span class="cx"> return true;
</span><span class="cx">
</span><span class="cx"> // Don't allow the attach if the window would be too small to accommodate the minimum inspector size.
</span><del>- unsigned inspectedPageHeight = m_inspectorController->inspectedPage()->mainFrame().view()->visibleHeight();
- unsigned inspectedPageWidth = m_inspectorController->inspectedPage()->mainFrame().view()->visibleWidth();
</del><ins>+ unsigned inspectedPageHeight = m_inspectorController->inspectedPage().mainFrame().view()->visibleHeight();
+ unsigned inspectedPageWidth = m_inspectorController->inspectedPage().mainFrame().view()->visibleWidth();
</ins><span class="cx"> unsigned maximumAttachedHeight = inspectedPageHeight * maximumAttachedHeightRatio;
</span><span class="cx"> return minimumAttachedHeight <= maximumAttachedHeight && minimumAttachedWidth <= inspectedPageWidth;
</span><span class="cx"> }
</span><span class="lines">@@ -195,7 +195,7 @@
</span><span class="cx">
</span><span class="cx"> void InspectorFrontendClientLocal::changeAttachedWindowHeight(unsigned height)
</span><span class="cx"> {
</span><del>- unsigned totalHeight = m_frontendPage->mainFrame().view()->visibleHeight() + m_inspectorController->inspectedPage()->mainFrame().view()->visibleHeight();
</del><ins>+ unsigned totalHeight = m_frontendPage->mainFrame().view()->visibleHeight() + m_inspectorController->inspectedPage().mainFrame().view()->visibleHeight();
</ins><span class="cx"> unsigned attachedHeight = constrainedAttachedWindowHeight(height, totalHeight);
</span><span class="cx"> m_settings->setProperty(inspectorAttachedHeightSetting, String::number(attachedHeight));
</span><span class="cx"> setAttachedWindowHeight(attachedHeight);
</span><span class="lines">@@ -203,7 +203,7 @@
</span><span class="cx">
</span><span class="cx"> void InspectorFrontendClientLocal::changeAttachedWindowWidth(unsigned width)
</span><span class="cx"> {
</span><del>- unsigned totalWidth = m_frontendPage->mainFrame().view()->visibleWidth() + m_inspectorController->inspectedPage()->mainFrame().view()->visibleWidth();
</del><ins>+ unsigned totalWidth = m_frontendPage->mainFrame().view()->visibleWidth() + m_inspectorController->inspectedPage().mainFrame().view()->visibleWidth();
</ins><span class="cx"> unsigned attachedWidth = constrainedAttachedWindowWidth(width, totalWidth);
</span><span class="cx"> setAttachedWindowWidth(attachedWidth);
</span><span class="cx"> }
</span><span class="lines">@@ -211,8 +211,7 @@
</span><span class="cx"> void InspectorFrontendClientLocal::openInNewTab(const String& url)
</span><span class="cx"> {
</span><span class="cx"> UserGestureIndicator indicator(DefinitelyProcessingUserGesture);
</span><del>- Page* page = m_inspectorController->inspectedPage();
- Frame& mainFrame = page->mainFrame();
</del><ins>+ Frame& mainFrame = m_inspectorController->inspectedPage().mainFrame();
</ins><span class="cx"> FrameLoadRequest request(mainFrame.document()->securityOrigin(), ResourceRequest(), "_blank");
</span><span class="cx">
</span><span class="cx"> bool created;
</span><span class="lines">@@ -257,7 +256,7 @@
</span><span class="cx">
</span><span class="cx"> void InspectorFrontendClientLocal::restoreAttachedWindowHeight()
</span><span class="cx"> {
</span><del>- unsigned inspectedPageHeight = m_inspectorController->inspectedPage()->mainFrame().view()->visibleHeight();
</del><ins>+ unsigned inspectedPageHeight = m_inspectorController->inspectedPage().mainFrame().view()->visibleHeight();
</ins><span class="cx"> String value = m_settings->getProperty(inspectorAttachedHeightSetting);
</span><span class="cx"> unsigned preferredHeight = value.isEmpty() ? defaultAttachedHeight : value.toUInt();
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorOverlaycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorOverlay.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorOverlay.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/InspectorOverlay.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -228,10 +228,8 @@
</span><span class="cx"> buildRendererHighlight(renderer, region, highlightConfig, highlight);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-static void buildQuadHighlight(Page* page, const FloatQuad& quad, const HighlightConfig& highlightConfig, Highlight *highlight)
</del><ins>+static void buildQuadHighlight(const FloatQuad& quad, const HighlightConfig& highlightConfig, Highlight *highlight)
</ins><span class="cx"> {
</span><del>- if (!page)
- return;
</del><span class="cx"> highlight->setDataFromConfig(highlightConfig);
</span><span class="cx"> highlight->type = HighlightTypeRects;
</span><span class="cx"> highlight->quads.append(quad);
</span><span class="lines">@@ -239,7 +237,7 @@
</span><span class="cx">
</span><span class="cx"> } // anonymous namespace
</span><span class="cx">
</span><del>-InspectorOverlay::InspectorOverlay(Page* page, InspectorClient* client)
</del><ins>+InspectorOverlay::InspectorOverlay(Page& page, InspectorClient* client)
</ins><span class="cx"> : m_page(page)
</span><span class="cx"> , m_client(client)
</span><span class="cx"> {
</span><span class="lines">@@ -274,7 +272,7 @@
</span><span class="cx"> if (m_highlightNode)
</span><span class="cx"> buildNodeHighlight(m_highlightNode.get(), nullptr, m_nodeHighlightConfig, highlight);
</span><span class="cx"> else
</span><del>- buildQuadHighlight(m_page, *m_highlightQuad, m_quadHighlightConfig, highlight);
</del><ins>+ buildQuadHighlight(*m_highlightQuad, m_quadHighlightConfig, highlight);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void InspectorOverlay::setPausedInDebuggerMessage(const String* message)
</span><span class="lines">@@ -300,7 +298,7 @@
</span><span class="cx"> void InspectorOverlay::highlightQuad(PassOwnPtr<FloatQuad> quad, const HighlightConfig& highlightConfig)
</span><span class="cx"> {
</span><span class="cx"> if (m_quadHighlightConfig.usePageCoordinates)
</span><del>- *quad -= m_page->mainFrame().view()->scrollOffset();
</del><ins>+ *quad -= m_page.mainFrame().view()->scrollOffset();
</ins><span class="cx">
</span><span class="cx"> m_quadHighlightConfig = highlightConfig;
</span><span class="cx"> m_highlightQuad = quad;
</span><span class="lines">@@ -324,7 +322,7 @@
</span><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- FrameView* view = m_page->mainFrame().view();
</del><ins>+ FrameView* view = m_page.mainFrame().view();
</ins><span class="cx"> if (!view)
</span><span class="cx"> return;
</span><span class="cx">
</span><span class="lines">@@ -332,8 +330,8 @@
</span><span class="cx"> IntSize viewportSize = view->visibleContentRect().size();
</span><span class="cx"> IntSize frameViewFullSize = view->visibleContentRect(ScrollableArea::IncludeScrollbars).size();
</span><span class="cx"> IntSize size = m_size.isEmpty() ? frameViewFullSize : m_size;
</span><del>- overlayPage()->setPageScaleFactor(m_page->pageScaleFactor(), IntPoint());
- size.scale(m_page->pageScaleFactor());
</del><ins>+ overlayPage()->setPageScaleFactor(m_page.pageScaleFactor(), IntPoint());
+ size.scale(m_page.pageScaleFactor());
</ins><span class="cx"> overlayView->resize(size);
</span><span class="cx">
</span><span class="cx"> // Clear canvas and paint things.
</span><span class="lines">@@ -710,7 +708,7 @@
</span><span class="cx"> RefPtr<InspectorObject> highlightObject = InspectorObject::create();
</span><span class="cx">
</span><span class="cx"> // The main view's scroll offset is shared across all quads.
</span><del>- FrameView* mainView = m_page->mainFrame().view();
</del><ins>+ FrameView* mainView = m_page.mainFrame().view();
</ins><span class="cx"> highlightObject->setObject("scroll", buildObjectForPoint(!mainView->delegatesScrolling() ? mainView->visibleContentRect().location() : FloatPoint()));
</span><span class="cx">
</span><span class="cx"> highlightObject->setArray("fragments", highlightFragments.release());
</span><span class="lines">@@ -738,7 +736,7 @@
</span><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> Highlight highlight;
</span><del>- buildQuadHighlight(m_page, *m_highlightQuad, m_quadHighlightConfig, &highlight);
</del><ins>+ buildQuadHighlight(*m_highlightQuad, m_quadHighlightConfig, &highlight);
</ins><span class="cx"> evaluateInOverlay("drawQuadHighlight", buildObjectForHighlight(highlight));
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -757,7 +755,7 @@
</span><span class="cx"> fillWithEmptyClients(pageClients);
</span><span class="cx"> m_overlayPage = adoptPtr(new Page(pageClients));
</span><span class="cx">
</span><del>- Settings& settings = m_page->settings();
</del><ins>+ Settings& settings = m_page.settings();
</ins><span class="cx"> Settings& overlaySettings = m_overlayPage->settings();
</span><span class="cx">
</span><span class="cx"> overlaySettings.setStandardFontFamily(settings.standardFontFamily());
</span><span class="lines">@@ -798,7 +796,7 @@
</span><span class="cx"> void InspectorOverlay::reset(const IntSize& viewportSize, const IntSize& frameViewFullSize)
</span><span class="cx"> {
</span><span class="cx"> RefPtr<InspectorObject> resetData = InspectorObject::create();
</span><del>- resetData->setNumber("deviceScaleFactor", m_page->deviceScaleFactor());
</del><ins>+ resetData->setNumber("deviceScaleFactor", m_page.deviceScaleFactor());
</ins><span class="cx"> resetData->setObject("viewportSize", buildObjectForSize(viewportSize));
</span><span class="cx"> resetData->setObject("frameViewFullSize", buildObjectForSize(frameViewFullSize));
</span><span class="cx"> evaluateInOverlay("reset", resetData.release());
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorOverlayh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorOverlay.h (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorOverlay.h        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/InspectorOverlay.h        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -105,10 +105,7 @@
</span><span class="cx"> class InspectorOverlay {
</span><span class="cx"> WTF_MAKE_FAST_ALLOCATED;
</span><span class="cx"> public:
</span><del>- static PassOwnPtr<InspectorOverlay> create(Page* page, InspectorClient* client)
- {
- return adoptPtr(new InspectorOverlay(page, client));
- }
</del><ins>+ InspectorOverlay(Page&, InspectorClient*);
</ins><span class="cx"> ~InspectorOverlay();
</span><span class="cx">
</span><span class="cx"> void update();
</span><span class="lines">@@ -130,8 +127,6 @@
</span><span class="cx">
</span><span class="cx"> void freePage();
</span><span class="cx"> private:
</span><del>- InspectorOverlay(Page*, InspectorClient*);
-
</del><span class="cx"> void drawGutter();
</span><span class="cx"> void drawNodeHighlight();
</span><span class="cx"> void drawQuadHighlight();
</span><span class="lines">@@ -141,7 +136,7 @@
</span><span class="cx"> void evaluateInOverlay(const String& method, const String& argument);
</span><span class="cx"> void evaluateInOverlay(const String& method, PassRefPtr<Inspector::InspectorValue> argument);
</span><span class="cx">
</span><del>- Page* m_page;
</del><ins>+ Page& m_page;
</ins><span class="cx"> InspectorClient* m_client;
</span><span class="cx"> String m_pausedInDebuggerMessage;
</span><span class="cx"> RefPtr<Node> m_highlightNode;
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInstrumentingAgentscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InstrumentingAgents.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InstrumentingAgents.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/InstrumentingAgents.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -105,16 +105,12 @@
</span><span class="cx"> InstrumentingAgents* instrumentationForPage(Page* page)
</span><span class="cx"> {
</span><span class="cx"> ASSERT(isMainThread());
</span><del>- if (InspectorController* controller = page->inspectorController())
- return controller->m_instrumentingAgents.get();
- return 0;
</del><ins>+ return page ? page->inspectorController().m_instrumentingAgents.get() : nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> InstrumentingAgents* instrumentationForWorkerGlobalScope(WorkerGlobalScope* workerGlobalScope)
</span><span class="cx"> {
</span><del>- if (WorkerInspectorController* controller = workerGlobalScope->workerInspectorController())
- return controller->m_instrumentingAgents.get();
- return 0;
</del><ins>+ return workerGlobalScope ? workerGlobalScope->workerInspectorController().m_instrumentingAgents.get() : nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorWorkerInspectorControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/WorkerInspectorController.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/WorkerInspectorController.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/WorkerInspectorController.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -55,7 +55,6 @@
</span><span class="cx"> #include "WorkerRuntimeAgent.h"
</span><span class="cx"> #include "WorkerThread.h"
</span><span class="cx"> #include <inspector/InspectorBackendDispatcher.h>
</span><del>-#include <wtf/PassOwnPtr.h>
</del><span class="cx">
</span><span class="cx"> using namespace Inspector;
</span><span class="cx">
</span><span class="lines">@@ -66,36 +65,37 @@
</span><span class="cx"> class PageInspectorProxy : public InspectorFrontendChannel {
</span><span class="cx"> WTF_MAKE_FAST_ALLOCATED;
</span><span class="cx"> public:
</span><del>- explicit PageInspectorProxy(WorkerGlobalScope* workerGlobalScope) : m_workerGlobalScope(workerGlobalScope) { }
</del><ins>+ explicit PageInspectorProxy(WorkerGlobalScope& workerGlobalScope)
+ : m_workerGlobalScope(workerGlobalScope) { }
</ins><span class="cx"> virtual ~PageInspectorProxy() { }
</span><span class="cx"> private:
</span><span class="cx"> virtual bool sendMessageToFrontend(const String& message)
</span><span class="cx"> {
</span><del>- m_workerGlobalScope->thread()->workerReportingProxy().postMessageToPageInspector(message);
</del><ins>+ m_workerGlobalScope.thread()->workerReportingProxy().postMessageToPageInspector(message);
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><del>- WorkerGlobalScope* m_workerGlobalScope;
</del><ins>+ WorkerGlobalScope& m_workerGlobalScope;
</ins><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> }
</span><span class="cx">
</span><del>-WorkerInspectorController::WorkerInspectorController(WorkerGlobalScope* workerGlobalScope)
</del><ins>+WorkerInspectorController::WorkerInspectorController(WorkerGlobalScope& workerGlobalScope)
</ins><span class="cx"> : m_workerGlobalScope(workerGlobalScope)
</span><span class="cx"> , m_instrumentingAgents(InstrumentingAgents::create(*this))
</span><span class="cx"> , m_injectedScriptManager(std::make_unique<PageInjectedScriptManager>(*this, PageInjectedScriptHost::create()))
</span><del>- , m_runtimeAgent(0)
</del><ins>+ , m_runtimeAgent(nullptr)
</ins><span class="cx"> {
</span><del>- auto runtimeAgent = std::make_unique<WorkerRuntimeAgent>(m_instrumentingAgents.get(), m_injectedScriptManager.get(), workerGlobalScope);
</del><ins>+ auto runtimeAgent = std::make_unique<WorkerRuntimeAgent>(m_instrumentingAgents.get(), m_injectedScriptManager.get(), &workerGlobalScope);
</ins><span class="cx"> m_runtimeAgent = runtimeAgent.get();
</span><span class="cx"> m_agents.append(std::move(runtimeAgent));
</span><span class="cx">
</span><span class="cx"> auto consoleAgent = std::make_unique<WorkerConsoleAgent>(m_instrumentingAgents.get(), m_injectedScriptManager.get());
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- auto debuggerAgent = std::make_unique<WorkerDebuggerAgent>(m_instrumentingAgents.get(), workerGlobalScope, m_injectedScriptManager.get());
</del><ins>+ auto debuggerAgent = std::make_unique<WorkerDebuggerAgent>(m_instrumentingAgents.get(), &workerGlobalScope, m_injectedScriptManager.get());
</ins><span class="cx"> m_runtimeAgent->setScriptDebugServer(&debuggerAgent->scriptDebugServer());
</span><span class="cx"> m_agents.append(std::move(debuggerAgent));
</span><span class="cx">
</span><del>- m_agents.append(InspectorProfilerAgent::create(m_instrumentingAgents.get(), consoleAgent.get(), workerGlobalScope, m_injectedScriptManager.get()));
</del><ins>+ m_agents.append(InspectorProfilerAgent::create(m_instrumentingAgents.get(), consoleAgent.get(), &workerGlobalScope, m_injectedScriptManager.get()));
</ins><span class="cx"> m_agents.append(std::make_unique<InspectorHeapProfilerAgent>(m_instrumentingAgents.get(), m_injectedScriptManager.get()));
</span><span class="cx"> #endif
</span><span class="cx"> m_agents.append(std::make_unique<InspectorTimelineAgent>(m_instrumentingAgents.get(), nullptr, nullptr, InspectorTimelineAgent::WorkerInspector, nullptr));
</span><span class="lines">@@ -122,7 +122,7 @@
</span><span class="cx"> void WorkerInspectorController::connectFrontend()
</span><span class="cx"> {
</span><span class="cx"> ASSERT(!m_frontendChannel);
</span><del>- m_frontendChannel = adoptPtr(new PageInspectorProxy(m_workerGlobalScope));
</del><ins>+ m_frontendChannel = std::make_unique<PageInspectorProxy>(m_workerGlobalScope);
</ins><span class="cx"> m_backendDispatcher = InspectorBackendDispatcher::create(m_frontendChannel.get());
</span><span class="cx"> m_agents.didCreateFrontendAndBackend(m_frontendChannel.get(), m_backendDispatcher.get());
</span><span class="cx"> }
</span><span class="lines">@@ -134,7 +134,7 @@
</span><span class="cx"> m_agents.willDestroyFrontendAndBackend();
</span><span class="cx"> m_backendDispatcher->clearFrontend();
</span><span class="cx"> m_backendDispatcher.clear();
</span><del>- m_frontendChannel.clear();
</del><ins>+ m_frontendChannel = nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WorkerInspectorController::dispatchMessageFromFrontend(const String& message)
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorWorkerInspectorControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/WorkerInspectorController.h (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/WorkerInspectorController.h        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/inspector/WorkerInspectorController.h        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -40,7 +40,6 @@
</span><span class="cx"> #include <wtf/FastMalloc.h>
</span><span class="cx"> #include <wtf/Forward.h>
</span><span class="cx"> #include <wtf/Noncopyable.h>
</span><del>-#include <wtf/OwnPtr.h>
</del><span class="cx"> #include <wtf/RefPtr.h>
</span><span class="cx"> #include <wtf/Vector.h>
</span><span class="cx">
</span><span class="lines">@@ -60,7 +59,7 @@
</span><span class="cx"> WTF_MAKE_NONCOPYABLE(WorkerInspectorController);
</span><span class="cx"> WTF_MAKE_FAST_ALLOCATED;
</span><span class="cx"> public:
</span><del>- WorkerInspectorController(WorkerGlobalScope*);
</del><ins>+ explicit WorkerInspectorController(WorkerGlobalScope&);
</ins><span class="cx"> ~WorkerInspectorController();
</span><span class="cx">
</span><span class="cx"> void connectFrontend();
</span><span class="lines">@@ -80,12 +79,12 @@
</span><span class="cx"> private:
</span><span class="cx"> friend InstrumentingAgents* instrumentationForWorkerGlobalScope(WorkerGlobalScope*);
</span><span class="cx">
</span><del>- WorkerGlobalScope* m_workerGlobalScope;
</del><ins>+ WorkerGlobalScope& m_workerGlobalScope;
</ins><span class="cx"> RefPtr<InstrumentingAgents> m_instrumentingAgents;
</span><span class="cx"> std::unique_ptr<PageInjectedScriptManager> m_injectedScriptManager;
</span><span class="cx"> InspectorRuntimeAgent* m_runtimeAgent;
</span><span class="cx"> Inspector::InspectorAgentRegistry m_agents;
</span><del>- OwnPtr<InspectorFrontendChannel> m_frontendChannel;
</del><ins>+ std::unique_ptr<InspectorFrontendChannel> m_frontendChannel;
</ins><span class="cx"> RefPtr<Inspector::InspectorBackendDispatcher> m_backendDispatcher;
</span><span class="cx"> Vector<InspectorInstrumentationCookie, 2> m_injectedScriptInstrumentationCookies;
</span><span class="cx"> };
</span></span></pre></div>
<a id="trunkSourceWebCoreloaderFrameLoadercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/loader/FrameLoader.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -2956,7 +2956,7 @@
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER) && ENABLE(INSPECTOR)
</span><span class="cx"> if (Page* page = m_frame.page()) {
</span><span class="cx"> if (m_frame.isMainFrame())
</span><del>- page->inspectorController()->resume();
</del><ins>+ page->inspectorController().resume();
</ins><span class="cx"> }
</span><span class="cx"> #endif
</span><span class="cx">
</span><span class="lines">@@ -3351,7 +3351,7 @@
</span><span class="cx">
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><span class="cx"> if (Page* page = m_frame.page())
</span><del>- page->inspectorController()->didClearWindowObjectInWorld(&m_frame, world);
</del><ins>+ page->inspectorController().didClearWindowObjectInWorld(&m_frame, world);
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> InspectorInstrumentation::didClearWindowObjectInWorld(&m_frame, world);
</span></span></pre></div>
<a id="trunkSourceWebCorepageContextMenuControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/ContextMenuController.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/ContextMenuController.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/page/ContextMenuController.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -157,7 +157,7 @@
</span><span class="cx"> void ContextMenuController::showContextMenu(Event* event)
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- if (m_page.inspectorController()->enabled())
</del><ins>+ if (m_page.inspectorController().enabled())
</ins><span class="cx"> addInspectElementItem();
</span><span class="cx"> #endif
</span><span class="cx">
</span><span class="lines">@@ -503,7 +503,7 @@
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><span class="cx"> case ContextMenuItemTagInspectElement:
</span><span class="cx"> if (Page* page = frame->page())
</span><del>- page->inspectorController()->inspect(m_hitTestResult.innerNonSharedNode());
</del><ins>+ page->inspectorController().inspect(m_hitTestResult.innerNonSharedNode());
</ins><span class="cx"> break;
</span><span class="cx"> #endif
</span><span class="cx"> case ContextMenuItemTagDictationAlternative:
</span><span class="lines">@@ -893,7 +893,7 @@
</span><span class="cx"> #endif
</span><span class="cx"> } else {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- if (!(frame->page() && (frame->page()->inspectorController()->hasInspectorFrontendClient() || frame->page()->inspectorController()->hasRemoteFrontend()))) {
</del><ins>+ if (!(frame->page() && (frame->page()->inspectorController().hasInspectorFrontendClient() || frame->page()->inspectorController().hasRemoteFrontend()))) {
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> // In GTK+ unavailable items are not hidden but insensitive.
</span><span class="lines">@@ -1101,9 +1101,6 @@
</span><span class="cx"> if (!page)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- if (!page->inspectorController())
- return;
-
</del><span class="cx"> ContextMenuItem InspectElementItem(ActionType, ContextMenuItemTagInspectElement, contextMenuItemTagInspectElement());
</span><span class="cx"> #if USE(CROSS_PLATFORM_CONTEXT_MENUS)
</span><span class="cx"> if (m_contextMenu && !m_contextMenu->items().isEmpty())
</span></span></pre></div>
<a id="trunkSourceWebCorepageFrameViewcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/FrameView.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/FrameView.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/page/FrameView.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -2938,7 +2938,7 @@
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><span class="cx"> Page* page = frame().page();
</span><span class="cx"> if (InspectorInstrumentation::hasFrontends() && isMainFrame) {
</span><del>- if (InspectorClient* inspectorClient = page ? page->inspectorController()->inspectorClient() : 0)
</del><ins>+ if (InspectorClient* inspectorClient = page ? page->inspectorController().inspectorClient() : nullptr)
</ins><span class="cx"> inspectorClient->didResizeMainFrame(&frame());
</span><span class="cx"> }
</span><span class="cx"> #endif
</span></span></pre></div>
<a id="trunkSourceWebCorepagePagecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/Page.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/Page.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/page/Page.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -134,7 +134,7 @@
</span><span class="cx"> , m_contextMenuController(std::make_unique<ContextMenuController>(*this, *pageClients.contextMenuClient))
</span><span class="cx"> #endif
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- , m_inspectorController(InspectorController::create(this, pageClients.inspectorClient))
</del><ins>+ , m_inspectorController(std::make_unique<InspectorController>(*this, pageClients.inspectorClient))
</ins><span class="cx"> #endif
</span><span class="cx"> #if ENABLE(POINTER_LOCK)
</span><span class="cx"> , m_pointerLockController(PointerLockController::create(this))
</span></span></pre></div>
<a id="trunkSourceWebCorepagePageh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/Page.h (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/Page.h        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/page/Page.h        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -202,7 +202,7 @@
</span><span class="cx"> ContextMenuController& contextMenuController() const { return *m_contextMenuController; }
</span><span class="cx"> #endif
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- InspectorController* inspectorController() const { return m_inspectorController.get(); }
</del><ins>+ InspectorController& inspectorController() const { return *m_inspectorController; }
</ins><span class="cx"> #endif
</span><span class="cx"> #if ENABLE(POINTER_LOCK)
</span><span class="cx"> PointerLockController* pointerLockController() const { return m_pointerLockController.get(); }
</span><span class="lines">@@ -456,7 +456,7 @@
</span><span class="cx"> const std::unique_ptr<ContextMenuController> m_contextMenuController;
</span><span class="cx"> #endif
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- OwnPtr<InspectorController> m_inspectorController;
</del><ins>+ const std::unique_ptr<InspectorController> m_inspectorController;
</ins><span class="cx"> #endif
</span><span class="cx"> #if ENABLE(POINTER_LOCK)
</span><span class="cx"> OwnPtr<PointerLockController> m_pointerLockController;
</span></span></pre></div>
<a id="trunkSourceWebCorepagePageDebuggablecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/PageDebuggable.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/PageDebuggable.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/page/PageDebuggable.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -63,12 +63,12 @@
</span><span class="cx">
</span><span class="cx"> bool PageDebuggable::hasLocalDebugger() const
</span><span class="cx"> {
</span><del>- return m_page.inspectorController()->hasLocalFrontend();
</del><ins>+ return m_page.inspectorController().hasLocalFrontend();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> pid_t PageDebuggable::parentProcessIdentifier() const
</span><span class="cx"> {
</span><del>- if (InspectorClient* inspectorClient = m_page.inspectorController()->inspectorClient())
</del><ins>+ if (InspectorClient* inspectorClient = m_page.inspectorController().inspectorClient())
</ins><span class="cx"> return inspectorClient->parentProcessIdentifier();
</span><span class="cx">
</span><span class="cx"> return 0;
</span><span class="lines">@@ -76,26 +76,26 @@
</span><span class="cx">
</span><span class="cx"> void PageDebuggable::connect(Inspector::InspectorFrontendChannel* channel)
</span><span class="cx"> {
</span><del>- InspectorController* inspectorController = m_page.inspectorController();
- inspectorController->setHasRemoteFrontend(true);
- inspectorController->connectFrontend(reinterpret_cast<WebCore::InspectorFrontendChannel*>(channel));
</del><ins>+ InspectorController& inspectorController = m_page.inspectorController();
+ inspectorController.setHasRemoteFrontend(true);
+ inspectorController.connectFrontend(reinterpret_cast<WebCore::InspectorFrontendChannel*>(channel));
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PageDebuggable::disconnect()
</span><span class="cx"> {
</span><del>- InspectorController* inspectorController = m_page.inspectorController();
- inspectorController->disconnectFrontend();
- inspectorController->setHasRemoteFrontend(false);
</del><ins>+ InspectorController& inspectorController = m_page.inspectorController();
+ inspectorController.disconnectFrontend();
+ inspectorController.setHasRemoteFrontend(false);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PageDebuggable::dispatchMessageFromRemoteFrontend(const String& message)
</span><span class="cx"> {
</span><del>- m_page.inspectorController()->dispatchMessageFromFrontend(message);
</del><ins>+ m_page.inspectorController().dispatchMessageFromFrontend(message);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PageDebuggable::setIndicating(bool indicating)
</span><span class="cx"> {
</span><del>- m_page.inspectorController()->setIndicating(indicating);
</del><ins>+ m_page.inspectorController().setIndicating(indicating);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformgraphicstexmapcoordinatedCompositingCoordinatorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/graphics/texmap/coordinated/CompositingCoordinator.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/graphics/texmap/coordinated/CompositingCoordinator.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/platform/graphics/texmap/coordinated/CompositingCoordinator.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -126,7 +126,7 @@
</span><span class="cx"> void CompositingCoordinator::syncDisplayState()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- m_page->inspectorController()->didBeginFrame();
</del><ins>+ m_page->inspectorController().didBeginFrame();
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> #if ENABLE(REQUEST_ANIMATION_FRAME) && !USE(REQUEST_ANIMATION_FRAME_TIMER) && !USE(REQUEST_ANIMATION_FRAME_DISPLAY_MONITOR)
</span></span></pre></div>
<a id="trunkSourceWebCoretestingInternalscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/testing/Internals.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/testing/Internals.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/testing/Internals.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -276,8 +276,7 @@
</span><span class="cx"> WebCore::overrideUserPreferredLanguages(Vector<String>());
</span><span class="cx"> WebCore::Settings::setUsesOverlayScrollbars(false);
</span><span class="cx"> #if ENABLE(INSPECTOR) && ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- if (page->inspectorController())
- page->inspectorController()->setProfilerEnabled(false);
</del><ins>+ page->inspectorController().setProfilerEnabled(false);
</ins><span class="cx"> #endif
</span><span class="cx"> #if ENABLE(VIDEO_TRACK) && !PLATFORM(WIN)
</span><span class="cx"> page->group().captionPreferences()->setCaptionsStyleSheetOverride(emptyString());
</span><span class="lines">@@ -704,13 +703,13 @@
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><span class="cx"> Document* document = contextDocument();
</span><del>- if (!document || !document->page() || !document->page()->inspectorController()) {
</del><ins>+ if (!document || !document->page()) {
</ins><span class="cx"> ec = INVALID_ACCESS_ERR;
</span><span class="cx"> return ClientRectList::create();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> Highlight highlight;
</span><del>- document->page()->inspectorController()->getHighlight(&highlight);
</del><ins>+ document->page()->inspectorController().getHighlight(&highlight);
</ins><span class="cx"> return ClientRectList::create(highlight.quads);
</span><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(ec);
</span><span class="lines">@@ -722,11 +721,11 @@
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><span class="cx"> Document* document = contextDocument();
</span><del>- if (!document || !document->page() || !document->page()->inspectorController()) {
</del><ins>+ if (!document || !document->page()) {
</ins><span class="cx"> ec = INVALID_ACCESS_ERR;
</span><span class="cx"> return String();
</span><span class="cx"> }
</span><del>- RefPtr<InspectorObject> object = document->page()->inspectorController()->buildObjectForHighlightedNode();
</del><ins>+ RefPtr<InspectorObject> object = document->page()->inspectorController().buildObjectForHighlightedNode();
</ins><span class="cx"> return object ? object->toJSONString() : String();
</span><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(ec);
</span><span class="lines">@@ -1153,16 +1152,14 @@
</span><span class="cx"> void Internals::emitInspectorDidBeginFrame()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- InspectorController* inspectorController = contextDocument()->frame()->page()->inspectorController();
- inspectorController->didBeginFrame();
</del><ins>+ contextDocument()->frame()->page()->inspectorController().didBeginFrame();
</ins><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void Internals::emitInspectorDidCancelFrame()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- InspectorController* inspectorController = contextDocument()->frame()->page()->inspectorController();
- inspectorController->didCancelFrame();
</del><ins>+ contextDocument()->frame()->page()->inspectorController().didCancelFrame();
</ins><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -1450,13 +1447,13 @@
</span><span class="cx"> Page* frontendPage = m_frontendWindow->document()->page();
</span><span class="cx"> ASSERT(frontendPage);
</span><span class="cx">
</span><del>- OwnPtr<InspectorFrontendClientDummy> frontendClient = adoptPtr(new InspectorFrontendClientDummy(page->inspectorController(), frontendPage));
</del><ins>+ auto frontendClient = std::make_unique<InspectorFrontendClientDummy>(&page->inspectorController(), frontendPage);
</ins><span class="cx">
</span><del>- frontendPage->inspectorController()->setInspectorFrontendClient(frontendClient.release());
</del><ins>+ frontendPage->inspectorController().setInspectorFrontendClient(std::move(frontendClient));
</ins><span class="cx">
</span><span class="cx"> m_frontendChannel = adoptPtr(new InspectorFrontendChannelDummy(frontendPage));
</span><span class="cx">
</span><del>- page->inspectorController()->connectFrontend(m_frontendChannel.get());
</del><ins>+ page->inspectorController().connectFrontend(m_frontendChannel.get());
</ins><span class="cx">
</span><span class="cx"> return m_frontendWindow;
</span><span class="cx"> }
</span><span class="lines">@@ -1467,7 +1464,7 @@
</span><span class="cx"> ASSERT(page);
</span><span class="cx"> ASSERT(m_frontendWindow);
</span><span class="cx">
</span><del>- page->inspectorController()->disconnectFrontend();
</del><ins>+ page->inspectorController().disconnectFrontend();
</ins><span class="cx">
</span><span class="cx"> m_frontendChannel.release();
</span><span class="cx">
</span><span class="lines">@@ -1478,22 +1475,22 @@
</span><span class="cx"> void Internals::setInspectorResourcesDataSizeLimits(int maximumResourcesContentSize, int maximumSingleResourceContentSize, ExceptionCode& ec)
</span><span class="cx"> {
</span><span class="cx"> Page* page = contextDocument()->frame()->page();
</span><del>- if (!page || !page->inspectorController()) {
</del><ins>+ if (!page) {
</ins><span class="cx"> ec = INVALID_ACCESS_ERR;
</span><span class="cx"> return;
</span><span class="cx"> }
</span><del>- page->inspectorController()->setResourcesDataSizeLimitsFromInternals(maximumResourcesContentSize, maximumSingleResourceContentSize);
</del><ins>+ page->inspectorController().setResourcesDataSizeLimitsFromInternals(maximumResourcesContentSize, maximumSingleResourceContentSize);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void Internals::setJavaScriptProfilingEnabled(bool enabled, ExceptionCode& ec)
</span><span class="cx"> {
</span><span class="cx"> Page* page = contextDocument()->frame()->page();
</span><del>- if (!page || !page->inspectorController()) {
</del><ins>+ if (!page) {
</ins><span class="cx"> ec = INVALID_ACCESS_ERR;
</span><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- page->inspectorController()->setProfilerEnabled(enabled);
</del><ins>+ page->inspectorController().setProfilerEnabled(enabled);
</ins><span class="cx"> }
</span><span class="cx"> #endif // ENABLE(INSPECTOR)
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreworkersWorkerGlobalScopecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/workers/WorkerGlobalScope.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/workers/WorkerGlobalScope.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/workers/WorkerGlobalScope.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -88,7 +88,7 @@
</span><span class="cx"> , m_script(adoptPtr(new WorkerScriptController(this)))
</span><span class="cx"> , m_thread(thread)
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- , m_workerInspectorController(adoptPtr(new WorkerInspectorController(this)))
</del><ins>+ , m_workerInspectorController(std::make_unique<WorkerInspectorController>(*this))
</ins><span class="cx"> #endif
</span><span class="cx"> , m_closing(false)
</span><span class="cx"> , m_eventQueue(*this)
</span><span class="lines">@@ -192,13 +192,6 @@
</span><span class="cx"> DOMTimer::removeById(scriptExecutionContext(), timeoutId);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-#if ENABLE(INSPECTOR)
-void WorkerGlobalScope::clearInspector()
-{
- m_workerInspectorController.clear();
-}
-#endif
-
</del><span class="cx"> int WorkerGlobalScope::setInterval(PassOwnPtr<ScheduledAction> action, int timeout)
</span><span class="cx"> {
</span><span class="cx"> return DOMTimer::install(scriptExecutionContext(), action, timeout, false);
</span></span></pre></div>
<a id="trunkSourceWebCoreworkersWorkerGlobalScopeh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/workers/WorkerGlobalScope.h (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/workers/WorkerGlobalScope.h        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/workers/WorkerGlobalScope.h        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -73,9 +73,6 @@
</span><span class="cx">
</span><span class="cx"> WorkerScriptController* script() { return m_script.get(); }
</span><span class="cx"> void clearScript() { m_script.clear(); }
</span><del>-#if ENABLE(INSPECTOR)
- void clearInspector();
-#endif
</del><span class="cx">
</span><span class="cx"> WorkerThread* thread() const { return m_thread; }
</span><span class="cx">
</span><span class="lines">@@ -106,7 +103,7 @@
</span><span class="cx"> virtual bool isJSExecutionForbidden() const override;
</span><span class="cx">
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- WorkerInspectorController* workerInspectorController() { return m_workerInspectorController.get(); }
</del><ins>+ WorkerInspectorController& workerInspectorController() { return *m_workerInspectorController; }
</ins><span class="cx"> #endif
</span><span class="cx"> // These methods are used for GC marking. See JSWorkerGlobalScope::visitChildrenVirtual(SlotVisitor&) in
</span><span class="cx"> // JSWorkerGlobalScopeCustom.cpp.
</span><span class="lines">@@ -168,7 +165,7 @@
</span><span class="cx"> WorkerThread* m_thread;
</span><span class="cx">
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- OwnPtr<WorkerInspectorController> m_workerInspectorController;
</del><ins>+ const std::unique_ptr<WorkerInspectorController> m_workerInspectorController;
</ins><span class="cx"> #endif
</span><span class="cx"> bool m_closing;
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreworkersWorkerMessagingProxycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/workers/WorkerMessagingProxy.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/workers/WorkerMessagingProxy.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/workers/WorkerMessagingProxy.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -409,7 +409,7 @@
</span><span class="cx"> static void connectToWorkerGlobalScopeInspectorTask(ScriptExecutionContext* context, bool)
</span><span class="cx"> {
</span><span class="cx"> ASSERT_WITH_SECURITY_IMPLICATION(context->isWorkerGlobalScope());
</span><del>- static_cast<WorkerGlobalScope*>(context)->workerInspectorController()->connectFrontend();
</del><ins>+ static_cast<WorkerGlobalScope*>(context)->workerInspectorController().connectFrontend();
</ins><span class="cx"> }
</span><span class="cx"> #endif
</span><span class="cx">
</span><span class="lines">@@ -428,7 +428,7 @@
</span><span class="cx"> static void disconnectFromWorkerGlobalScopeInspectorTask(ScriptExecutionContext* context, bool)
</span><span class="cx"> {
</span><span class="cx"> ASSERT_WITH_SECURITY_IMPLICATION(context->isWorkerGlobalScope());
</span><del>- static_cast<WorkerGlobalScope*>(context)->workerInspectorController()->disconnectFrontend();
</del><ins>+ static_cast<WorkerGlobalScope*>(context)->workerInspectorController().disconnectFrontend();
</ins><span class="cx"> }
</span><span class="cx"> #endif
</span><span class="cx">
</span><span class="lines">@@ -446,7 +446,7 @@
</span><span class="cx"> static void dispatchOnInspectorBackendTask(ScriptExecutionContext* context, const String& message)
</span><span class="cx"> {
</span><span class="cx"> ASSERT_WITH_SECURITY_IMPLICATION(context->isWorkerGlobalScope());
</span><del>- static_cast<WorkerGlobalScope*>(context)->workerInspectorController()->dispatchMessageFromFrontend(message);
</del><ins>+ static_cast<WorkerGlobalScope*>(context)->workerInspectorController().dispatchMessageFromFrontend(message);
</ins><span class="cx"> }
</span><span class="cx"> #endif
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreworkersWorkerThreadcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/workers/WorkerThread.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/workers/WorkerThread.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebCore/workers/WorkerThread.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -216,9 +216,6 @@
</span><span class="cx"> {
</span><span class="cx"> ASSERT_WITH_SECURITY_IMPLICATION(context->isWorkerGlobalScope());
</span><span class="cx"> WorkerGlobalScope* workerGlobalScope = static_cast<WorkerGlobalScope*>(context);
</span><del>-#if ENABLE(INSPECTOR)
- workerGlobalScope->clearInspector();
-#endif
</del><span class="cx"> // It's not safe to call clearScript until all the cleanup tasks posted by functions initiated by WorkerThreadShutdownStartTask have completed.
</span><span class="cx"> workerGlobalScope->clearScript();
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebKiteflChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/efl/ChangeLog (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/efl/ChangeLog        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/efl/ChangeLog        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -1,3 +1,21 @@
</span><ins>+2014-01-18 Brian Burg <bburg@apple.com>
+
+ Web Inspector: Page should use std::unique_ptr for InspectorController
+ https://bugs.webkit.org/show_bug.cgi?id=127068
+
+ Reviewed by Joseph Pecoraro.
+
+ Convert call sites to use a InspectorController reference.
+ Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
+
+ * WebCoreSupport/DumpRenderTreeSupportEfl.cpp:
+ (DumpRenderTreeSupportEfl::evaluateInWebInspector):
+ * WebCoreSupport/InspectorClientEfl.cpp:
+ (WebCore::InspectorClientEfl::openInspectorFrontend):
+ * ewk/ewk_view.cpp:
+ (ewk_view_inspector_show):
+ (ewk_view_inspector_close):
+
</ins><span class="cx"> 2014-01-17 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><span class="cx"> Remove another unused FrameLoaderClient callback
</span></span></pre></div>
<a id="trunkSourceWebKiteflWebCoreSupportDumpRenderTreeSupportEflcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -466,8 +466,7 @@
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><span class="cx"> DRT_SUPPRT_PAGE_GET_OR_RETURN(ewkView, page);
</span><span class="cx">
</span><del>- if (page->inspectorController())
- page->inspectorController()->evaluateForTestInFrontend(callId, script);
</del><ins>+ page->inspectorController().evaluateForTestInFrontend(callId, script);
</ins><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(ewkView);
</span><span class="cx"> UNUSED_PARAM(callId);
</span></span></pre></div>
<a id="trunkSourceWebKiteflWebCoreSupportInspectorClientEflcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -99,11 +99,11 @@
</span><span class="cx"> String inspectorUri = inspectorFilesPath() + "/Main.html";
</span><span class="cx"> ewk_view_uri_set(m_inspectorView, inspectorUri.utf8().data());
</span><span class="cx">
</span><del>- OwnPtr<InspectorFrontendClientEfl> frontendClient = adoptPtr(new InspectorFrontendClientEfl(m_inspectedView, m_inspectorView, this));
</del><ins>+ auto frontendClient = std::make_unique<InspectorFrontendClientEfl>(m_inspectedView, m_inspectorView, this);
</ins><span class="cx"> m_frontendClient = frontendClient.get();
</span><span class="cx">
</span><del>- InspectorController* controller = EWKPrivate::corePage(m_inspectorView)->inspectorController();
- controller->setInspectorFrontendClient(frontendClient.release());
</del><ins>+ InspectorController& controller = EWKPrivate::corePage(m_inspectorView)->inspectorController();
+ controller.setInspectorFrontendClient(std::move(frontendClient));
</ins><span class="cx">
</span><span class="cx"> return this;
</span><span class="cx"> }
</span><span class="lines">@@ -147,7 +147,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> InspectorFrontendClientEfl::InspectorFrontendClientEfl(Evas_Object* inspectedView, Evas_Object* inspectorView, InspectorClientEfl* inspectorClient)
</span><del>- : InspectorFrontendClientLocal(EWKPrivate::corePage(inspectedView)->inspectorController(), EWKPrivate::corePage(inspectorView), adoptPtr(new InspectorFrontendSettingsEfl()))
</del><ins>+ : InspectorFrontendClientLocal(&EWKPrivate::corePage(inspectedView)->inspectorController(), EWKPrivate::corePage(inspectorView), adoptPtr(new InspectorFrontendSettingsEfl()))
</ins><span class="cx"> , m_inspectedView(inspectedView)
</span><span class="cx"> , m_inspectorView(inspectorView)
</span><span class="cx"> , m_inspectorClient(inspectorClient)
</span><span class="lines">@@ -213,7 +213,7 @@
</span><span class="cx"> void InspectorFrontendClientEfl::destroyInspectorWindow(bool notifyInspectorController)
</span><span class="cx"> {
</span><span class="cx"> if (notifyInspectorController)
</span><del>- EWKPrivate::corePage(m_inspectedView)->inspectorController()->disconnectFrontend();
</del><ins>+ EWKPrivate::corePage(m_inspectedView)->inspectorController().disconnectFrontend();
</ins><span class="cx">
</span><span class="cx"> if (m_inspectorClient)
</span><span class="cx"> m_inspectorClient->releaseFrontendPage();
</span></span></pre></div>
<a id="trunkSourceWebKiteflewkewk_viewcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/efl/ewk/ewk_view.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/efl/ewk/ewk_view.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/efl/ewk/ewk_view.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -4662,7 +4662,7 @@
</span><span class="cx"> EWK_VIEW_SD_GET_OR_RETURN(ewkView, smartData);
</span><span class="cx"> EWK_VIEW_PRIV_GET_OR_RETURN(smartData, priv);
</span><span class="cx">
</span><del>- priv->page->inspectorController()->show();
</del><ins>+ priv->page->inspectorController().show();
</ins><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(ewkView);
</span><span class="cx"> #endif
</span><span class="lines">@@ -4674,7 +4674,7 @@
</span><span class="cx"> EWK_VIEW_SD_GET_OR_RETURN(ewkView, smartData);
</span><span class="cx"> EWK_VIEW_PRIV_GET_OR_RETURN(smartData, priv);
</span><span class="cx">
</span><del>- priv->page->inspectorController()->close();
</del><ins>+ priv->page->inspectorController().close();
</ins><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(ewkView);
</span><span class="cx"> #endif
</span></span></pre></div>
<a id="trunkSourceWebKitgtkChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/gtk/ChangeLog (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/gtk/ChangeLog        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/gtk/ChangeLog        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -1,3 +1,26 @@
</span><ins>+2014-01-18 Brian Burg <bburg@apple.com>
+
+ Web Inspector: Page should use std::unique_ptr for InspectorController
+ https://bugs.webkit.org/show_bug.cgi?id=127068
+
+ Reviewed by Joseph Pecoraro.
+
+ Convert call sites to use a InspectorController reference.
+ Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
+
+ * WebCoreSupport/ChromeClientGtk.cpp:
+ (WebKit::paintWebView):
+ * WebCoreSupport/InspectorClientGtk.cpp:
+ (WebKit::InspectorClient::openInspectorFrontend):
+ * webkit/webkitwebinspector.cpp:
+ (webkit_web_inspector_set_property):
+ (webkit_web_inspector_get_property):
+ (webkit_web_inspector_show):
+ (webkit_web_inspector_inspect_node):
+ (webkit_web_inspector_inspect_coordinates):
+ (webkit_web_inspector_close):
+ (webkit_web_inspector_execute_script):
+
</ins><span class="cx"> 2014-01-17 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><span class="cx"> Remove another unused FrameLoaderClient callback
</span></span></pre></div>
<a id="trunkSourceWebKitgtkWebCoreSupportChromeClientGtkcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -567,7 +567,7 @@
</span><span class="cx">
</span><span class="cx"> gc.save();
</span><span class="cx"> gc.clip(dirtyRegion.bounds());
</span><del>- frame->page()->inspectorController()->drawHighlight(gc);
</del><ins>+ frame->page()->inspectorController().drawHighlight(gc);
</ins><span class="cx"> gc.restore();
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebKitgtkWebCoreSupportInspectorClientGtkcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -109,9 +109,9 @@
</span><span class="cx"> gtk_widget_show(GTK_WIDGET(inspectorWebView));
</span><span class="cx">
</span><span class="cx"> m_frontendPage = core(inspectorWebView);
</span><del>- OwnPtr<InspectorFrontendClient> frontendClient = adoptPtr(new InspectorFrontendClient(m_inspectedWebView, inspectorWebView, webInspector, m_frontendPage, this));
</del><ins>+ auto frontendClient = std::make_unique<InspectorFrontendClient>(m_inspectedWebView, inspectorWebView, webInspector, m_frontendPage, this);
</ins><span class="cx"> m_frontendClient = frontendClient.get();
</span><del>- m_frontendPage->inspectorController()->setInspectorFrontendClient(frontendClient.release());
</del><ins>+ m_frontendPage->inspectorController().setInspectorFrontendClient(std::move(frontendClient));
</ins><span class="cx">
</span><span class="cx"> // The inspector must be in it's own PageGroup to avoid deadlock while debugging.
</span><span class="cx"> m_frontendPage->setGroupName("");
</span><span class="lines">@@ -156,7 +156,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> InspectorFrontendClient::InspectorFrontendClient(WebKitWebView* inspectedWebView, WebKitWebView* inspectorWebView, WebKitWebInspector* webInspector, Page* inspectorPage, InspectorClient* inspectorClient)
</span><del>- : InspectorFrontendClientLocal(core(inspectedWebView)->inspectorController(), inspectorPage, adoptPtr(new InspectorFrontendSettingsGtk()))
</del><ins>+ : InspectorFrontendClientLocal(&core(inspectedWebView)->inspectorController(), inspectorPage, adoptPtr(new InspectorFrontendSettingsGtk()))
</ins><span class="cx"> , m_inspectorWebView(inspectorWebView)
</span><span class="cx"> , m_inspectedWebView(inspectedWebView)
</span><span class="cx"> , m_webInspector(webInspector)
</span><span class="lines">@@ -189,7 +189,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (notifyInspectorController)
</span><del>- core(m_inspectedWebView)->inspectorController()->disconnectFrontend();
</del><ins>+ core(m_inspectedWebView)->inspectorController().disconnectFrontend();
</ins><span class="cx">
</span><span class="cx"> if (m_inspectorClient)
</span><span class="cx"> m_inspectorClient->releaseFrontendPage();
</span></span></pre></div>
<a id="trunkSourceWebKitgtkwebkitwebkitwebinspectorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/gtk/webkit/webkitwebinspector.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/gtk/webkit/webkitwebinspector.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/gtk/webkit/webkitwebinspector.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -353,7 +353,7 @@
</span><span class="cx"> case PROP_JAVASCRIPT_PROFILING_ENABLED: {
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><span class="cx"> bool enabled = g_value_get_boolean(value);
</span><del>- priv->page->inspectorController()->setProfilerEnabled(enabled);
</del><ins>+ priv->page->inspectorController().setProfilerEnabled(enabled);
</ins><span class="cx"> #else
</span><span class="cx"> g_message("PROP_JAVASCRIPT_PROFILING_ENABLED is not work because of the javascript debugger is disabled\n");
</span><span class="cx"> #endif
</span><span class="lines">@@ -383,7 +383,7 @@
</span><span class="cx"> break;
</span><span class="cx"> case PROP_JAVASCRIPT_PROFILING_ENABLED:
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- g_value_set_boolean(value, priv->page->inspectorController()->profilerEnabled());
</del><ins>+ g_value_set_boolean(value, priv->page->inspectorController().profilerEnabled());
</ins><span class="cx"> #else
</span><span class="cx"> g_message("PROP_JAVASCRIPT_PROFILING_ENABLED is not work because of the javascript debugger is disabled\n");
</span><span class="cx"> #endif
</span><span class="lines">@@ -491,7 +491,7 @@
</span><span class="cx"> if (!view)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- priv->page->inspectorController()->show();
</del><ins>+ priv->page->inspectorController().show();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> /**
</span><span class="lines">@@ -508,7 +508,7 @@
</span><span class="cx"> g_return_if_fail(WEBKIT_IS_WEB_INSPECTOR(webInspector));
</span><span class="cx"> g_return_if_fail(WEBKIT_DOM_IS_NODE(node));
</span><span class="cx">
</span><del>- webInspector->priv->page->inspectorController()->inspect(core(node));
</del><ins>+ webInspector->priv->page->inspectorController().inspect(core(node));
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> /**
</span><span class="lines">@@ -546,7 +546,7 @@
</span><span class="cx"> HitTestResult result(documentPoint);
</span><span class="cx">
</span><span class="cx"> frame.contentRenderer()->layer()->hitTest(request, result);
</span><del>- priv->page->inspectorController()->inspect(result.innerNonSharedNode());
</del><ins>+ priv->page->inspectorController().inspect(result.innerNonSharedNode());
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> /**
</span><span class="lines">@@ -562,7 +562,7 @@
</span><span class="cx"> g_return_if_fail(WEBKIT_IS_WEB_INSPECTOR(webInspector));
</span><span class="cx">
</span><span class="cx"> WebKitWebInspectorPrivate* priv = webInspector->priv;
</span><del>- priv->page->inspectorController()->close();
</del><ins>+ priv->page->inspectorController().close();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void webkit_web_inspector_execute_script(WebKitWebInspector* webInspector, long callId, const gchar* script)
</span><span class="lines">@@ -571,5 +571,5 @@
</span><span class="cx"> g_return_if_fail(script);
</span><span class="cx">
</span><span class="cx"> WebKitWebInspectorPrivate* priv = webInspector->priv;
</span><del>- priv->page->inspectorController()->evaluateForTestInFrontend(callId, script);
</del><ins>+ priv->page->inspectorController().evaluateForTestInFrontend(callId, script);
</ins><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebKitmacChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/ChangeLog (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/ChangeLog        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/mac/ChangeLog        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -1,3 +1,25 @@
</span><ins>+2014-01-18 Brian Burg <bburg@apple.com>
+
+ Web Inspector: Page should use std::unique_ptr for InspectorController
+ https://bugs.webkit.org/show_bug.cgi?id=127068
+
+ Reviewed by Joseph Pecoraro.
+
+ Convert call sites to use a InspectorController reference.
+ Convert instantiations of InspectorFrontendClient to use std::unique_ptr.
+
+ * WebCoreSupport/WebInspectorClient.mm:
+ (WebInspectorClient::openInspectorFrontend):
+ (-[WebInspectorWindowController destroyInspectorView:]):
+ * WebInspector/WebInspector.mm:
+ (-[WebInspector showWindow]):
+ (-[WebInspector isJavaScriptProfilingEnabled]):
+ (-[WebInspector setJavaScriptProfilingEnabled:]):
+ (-[WebInspector close:]):
+ (-[WebInspector evaluateInFrontend:callId:script:]):
+ * WebInspector/WebNodeHighlighter.mm:
+ (-[WebNodeHighlighter highlight]):
+
</ins><span class="cx"> 2014-01-18 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><span class="cx"> Remove ENABLE_THREADED_HTML_PARSER defines everywhere
</span></span></pre></div>
<a id="trunkSourceWebKitmacWebCoreSupportWebInspectorClientmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -145,11 +145,11 @@
</span><span class="cx"> [windowController.get() setInspectorClient:this];
</span><span class="cx">
</span><span class="cx"> m_frontendPage = core([windowController.get() webView]);
</span><del>- OwnPtr<WebInspectorFrontendClient> frontendClient = adoptPtr(new WebInspectorFrontendClient(m_webView, windowController.get(), inspectorController, m_frontendPage, createFrontendSettings()));
</del><ins>+ auto frontendClient = std::make_unique<WebInspectorFrontendClient>(m_webView, windowController.get(), inspectorController, m_frontendPage, createFrontendSettings());
</ins><span class="cx"> m_frontendClient = frontendClient.get();
</span><span class="cx"> RetainPtr<WebInspectorFrontend> webInspectorFrontend = adoptNS([[WebInspectorFrontend alloc] initWithFrontendClient:frontendClient.get()]);
</span><span class="cx"> [[m_webView inspector] setFrontend:webInspectorFrontend.get()];
</span><del>- m_frontendPage->inspectorController()->setInspectorFrontendClient(frontendClient.release());
</del><ins>+ m_frontendPage->inspectorController().setInspectorFrontendClient(std::move(frontendClient));
</ins><span class="cx"> return this;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -693,7 +693,7 @@
</span><span class="cx">
</span><span class="cx"> if (notifyInspectorController) {
</span><span class="cx"> if (Page* inspectedPage = [_inspectedWebView.get() page])
</span><del>- inspectedPage->inspectorController()->disconnectFrontend();
</del><ins>+ inspectedPage->inspectorController().disconnectFrontend();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> RetainPtr<WebInspectorWindowController> protect(self);
</span></span></pre></div>
<a id="trunkSourceWebKitmacWebInspectorWebInspectormm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/WebInspector/WebInspector.mm (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/WebInspector/WebInspector.mm        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/mac/WebInspector/WebInspector.mm        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -66,7 +66,7 @@
</span><span class="cx"> - (void)showWindow
</span><span class="cx"> {
</span><span class="cx"> if (Page* page = core(_webView))
</span><del>- page->inspectorController()->show();
</del><ins>+ page->inspectorController().show();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> - (void)show:(id)sender
</span><span class="lines">@@ -142,7 +142,7 @@
</span><span class="cx"> - (BOOL)isJavaScriptProfilingEnabled
</span><span class="cx"> {
</span><span class="cx"> if (Page* page = core(_webView))
</span><del>- return page->inspectorController()->profilerEnabled();
</del><ins>+ return page->inspectorController().profilerEnabled();
</ins><span class="cx"> return NO;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -152,7 +152,7 @@
</span><span class="cx"> if (!page)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- page->inspectorController()->setProfilerEnabled(enabled);
</del><ins>+ page->inspectorController().setProfilerEnabled(enabled);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> - (BOOL)isTimelineProfilingEnabled
</span><span class="lines">@@ -169,7 +169,7 @@
</span><span class="cx"> - (void)close:(id)sender
</span><span class="cx"> {
</span><span class="cx"> if (Page* page = core(_webView))
</span><del>- page->inspectorController()->close();
</del><ins>+ page->inspectorController().close();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> - (void)attach:(id)sender
</span><span class="lines">@@ -185,7 +185,7 @@
</span><span class="cx"> - (void)evaluateInFrontend:(id)sender callId:(long)callId script:(NSString *)script
</span><span class="cx"> {
</span><span class="cx"> if (Page* page = core(_webView))
</span><del>- page->inspectorController()->evaluateForTestInFrontend(callId, script);
</del><ins>+ page->inspectorController().evaluateForTestInFrontend(callId, script);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> - (void)setFrontend:(WebInspectorFrontend *)frontend
</span></span></pre></div>
<a id="trunkSourceWebKitmacWebInspectorWebNodeHighlightermm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/WebInspector/WebNodeHighlighter.mm (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/WebInspector/WebNodeHighlighter.mm        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/mac/WebInspector/WebNodeHighlighter.mm        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -66,7 +66,7 @@
</span><span class="cx"> return; // Skip the highlight if we have no window (e.g. hidden tab).
</span><span class="cx">
</span><span class="cx"> if (!_currentHighlight) {
</span><del>- _currentHighlight = [[WebNodeHighlight alloc] initWithTargetView:view inspectorController:[_inspectedWebView page]->inspectorController()];
</del><ins>+ _currentHighlight = [[WebNodeHighlight alloc] initWithTargetView:view inspectorController:&[_inspectedWebView page]->inspectorController()];
</ins><span class="cx"> [_currentHighlight setDelegate:self];
</span><span class="cx"> [_currentHighlight attach];
</span><span class="cx"> } else {
</span></span></pre></div>
<a id="trunkSourceWebKitwinChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/ChangeLog (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/ChangeLog        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/win/ChangeLog        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -1,3 +1,25 @@
</span><ins>+2014-01-18 Brian Burg <bburg@apple.com>
+
+ Web Inspector: Page should use std::unique_ptr for InspectorController
+ https://bugs.webkit.org/show_bug.cgi?id=127068
+
+ Reviewed by Joseph Pecoraro.
+
+ Convert call sites to use a InspectorController reference.
+
+ * WebCoreSupport/WebInspectorClient.cpp:
+ (WebInspectorClient::openInspectorFrontend):
+ (WebInspectorFrontendClient::destroyInspectorView):
+ (WebInspectorFrontendClient::onClose):
+ * WebInspector.cpp:
+ (WebInspector::show):
+ (WebInspector::close):
+ (WebInspector::isJavaScriptProfilingEnabled):
+ (WebInspector::setJavaScriptProfilingEnabled):
+ (WebInspector::evaluateInFrontend):
+ * WebNodeHighlight.cpp:
+ (WebNodeHighlight::update):
+
</ins><span class="cx"> 2014-01-17 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><span class="cx"> Clean up PageCache classes
</span></span></pre></div>
<a id="trunkSourceWebKitwinWebCoreSupportWebInspectorClientcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/WebCoreSupport/WebInspectorClient.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/WebCoreSupport/WebInspectorClient.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/win/WebCoreSupport/WebInspectorClient.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -170,9 +170,9 @@
</span><span class="cx"> return 0;
</span><span class="cx">
</span><span class="cx"> m_frontendPage = core(frontendWebView.get());
</span><del>- OwnPtr<WebInspectorFrontendClient> frontendClient = adoptPtr(new WebInspectorFrontendClient(m_inspectedWebView, m_inspectedWebViewHwnd, frontendHwnd, frontendWebView, frontendWebViewHwnd, this, createFrontendSettings()));
</del><ins>+ auto frontendClient = std::make_unique<WebInspectorFrontendClient>(m_inspectedWebView, m_inspectedWebViewHwnd, frontendHwnd, frontendWebView, frontendWebViewHwnd, this, createFrontendSettings());
</ins><span class="cx"> m_frontendClient = frontendClient.get();
</span><del>- m_frontendPage->inspectorController()->setInspectorFrontendClient(frontendClient.release());
</del><ins>+ m_frontendPage->inspectorController().setInspectorFrontendClient(std::move(frontendClient));
</ins><span class="cx"> m_frontendHwnd = frontendHwnd;
</span><span class="cx"> return this;
</span><span class="cx"> }
</span><span class="lines">@@ -224,7 +224,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> WebInspectorFrontendClient::WebInspectorFrontendClient(WebView* inspectedWebView, HWND inspectedWebViewHwnd, HWND frontendHwnd, const COMPtr<WebView>& frontendWebView, HWND frontendWebViewHwnd, WebInspectorClient* inspectorClient, PassOwnPtr<Settings> settings)
</span><del>- : InspectorFrontendClientLocal(inspectedWebView->page()->inspectorController(), core(frontendWebView.get()), settings)
</del><ins>+ : InspectorFrontendClientLocal(&inspectedWebView->page()->inspectorController(), core(frontendWebView.get()), settings)
</ins><span class="cx"> , m_inspectedWebView(inspectedWebView)
</span><span class="cx"> , m_inspectedWebViewHwnd(inspectedWebViewHwnd)
</span><span class="cx"> , m_inspectorClient(inspectorClient)
</span><span class="lines">@@ -438,7 +438,7 @@
</span><span class="cx"> closeWindowWithoutNotifications();
</span><span class="cx">
</span><span class="cx"> if (notifyInspectorController) {
</span><del>- m_inspectedWebView->page()->inspectorController()->disconnectFrontend();
</del><ins>+ m_inspectedWebView->page()->inspectorController().disconnectFrontend();
</ins><span class="cx"> m_inspectorClient->updateHighlight();
</span><span class="cx"> }
</span><span class="cx"> ::DestroyWindow(m_frontendHwnd);
</span><span class="lines">@@ -472,7 +472,7 @@
</span><span class="cx"> LRESULT WebInspectorFrontendClient::onClose(WPARAM, LPARAM)
</span><span class="cx"> {
</span><span class="cx"> ::ShowWindow(m_frontendHwnd, SW_HIDE);
</span><del>- m_inspectedWebView->page()->inspectorController()->close();
</del><ins>+ m_inspectedWebView->page()->inspectorController().close();
</ins><span class="cx">
</span><span class="cx"> return 0;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebKitwinWebInspectorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/WebInspector.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/WebInspector.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/win/WebInspector.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -109,7 +109,7 @@
</span><span class="cx"> {
</span><span class="cx"> if (m_webView)
</span><span class="cx"> if (Page* page = m_webView->page())
</span><del>- page->inspectorController()->show();
</del><ins>+ page->inspectorController().show();
</ins><span class="cx">
</span><span class="cx"> return S_OK;
</span><span class="cx"> }
</span><span class="lines">@@ -131,7 +131,7 @@
</span><span class="cx"> {
</span><span class="cx"> if (m_webView)
</span><span class="cx"> if (Page* page = m_webView->page())
</span><del>- page->inspectorController()->close();
</del><ins>+ page->inspectorController().close();
</ins><span class="cx">
</span><span class="cx"> return S_OK;
</span><span class="cx"> }
</span><span class="lines">@@ -219,7 +219,7 @@
</span><span class="cx"> if (!page)
</span><span class="cx"> return S_OK;
</span><span class="cx">
</span><del>- *isProfilingEnabled = page->inspectorController()->profilerEnabled();
</del><ins>+ *isProfilingEnabled = page->inspectorController().profilerEnabled();
</ins><span class="cx"> return S_OK;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -232,7 +232,7 @@
</span><span class="cx"> if (!page)
</span><span class="cx"> return S_OK;
</span><span class="cx">
</span><del>- page->inspectorController()->setProfilerEnabled(enabled);
</del><ins>+ page->inspectorController().setProfilerEnabled(enabled);
</ins><span class="cx">
</span><span class="cx"> return S_OK;
</span><span class="cx"> }
</span><span class="lines">@@ -247,7 +247,7 @@
</span><span class="cx"> return S_OK;
</span><span class="cx">
</span><span class="cx"> String script(bScript, SysStringLen(bScript));
</span><del>- page->inspectorController()->evaluateForTestInFrontend(callId, script);
</del><ins>+ page->inspectorController().evaluateForTestInFrontend(callId, script);
</ins><span class="cx"> return S_OK;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebKitwinWebNodeHighlightcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/WebNodeHighlight.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/WebNodeHighlight.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit/win/WebNodeHighlight.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -165,8 +165,8 @@
</span><span class="cx">
</span><span class="cx"> GraphicsContext context(hdc.get());
</span><span class="cx"> #if ENABLE(INSPECTOR)
</span><del>- m_inspectedWebView->page()->inspectorController()->drawHighlight(context);
-#endif // ENABLE(INSPECTOR)
</del><ins>+ m_inspectedWebView->page()->inspectorController().drawHighlight(context);
+#endif
</ins><span class="cx">
</span><span class="cx"> BLENDFUNCTION bf;
</span><span class="cx"> bf.BlendOp = AC_SRC_OVER;
</span></span></pre></div>
<a id="trunkSourceWebKit2ChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/ChangeLog (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/ChangeLog        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit2/ChangeLog        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -1,3 +1,36 @@
</span><ins>+2014-01-18 Brian Burg <bburg@apple.com>
+
+ Web Inspector: Page should use std::unique_ptr for InspectorController
+ https://bugs.webkit.org/show_bug.cgi?id=127068
+
+ Reviewed by Joseph Pecoraro.
+
+ Convert call sites to use a InspectorController reference.
+
+ * WebProcess/WebCoreSupport/WebInspectorClient.cpp:
+ (WebKit::WebInspectorClient::drawRect):
+ * WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp:
+ (WebKit::WebInspectorFrontendClient::WebInspectorFrontendClient):
+ (WebKit::WebInspectorFrontendClient::closeWindow):
+ * WebProcess/WebPage/WebInspector.cpp:
+ (WebKit::WebInspector::createInspectorPage):
+ (WebKit::WebInspector::show):
+ (WebKit::WebInspector::close):
+ (WebKit::WebInspector::evaluateScriptForTest):
+ (WebKit::WebInspector::showConsole):
+ (WebKit::WebInspector::showResources):
+ (WebKit::WebInspector::showMainResourceForFrame):
+ (WebKit::WebInspector::startJavaScriptDebugging):
+ (WebKit::WebInspector::stopJavaScriptDebugging):
+ (WebKit::WebInspector::setJavaScriptProfilingEnabled):
+ (WebKit::WebInspector::startJavaScriptProfiling):
+ (WebKit::WebInspector::stopJavaScriptProfiling):
+ (WebKit::WebInspector::startPageProfiling):
+ (WebKit::WebInspector::stopPageProfiling):
+ (WebKit::WebInspector::dispatchMessageFromRemoteFrontend):
+ (WebKit::WebInspector::remoteFrontendConnected):
+ (WebKit::WebInspector::remoteFrontendDisconnected):
+
</ins><span class="cx"> 2014-01-18 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><span class="cx"> Remove ENABLE_THREADED_HTML_PARSER defines everywhere
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebCoreSupportWebInspectorClientcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorClient.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorClient.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorClient.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -145,7 +145,7 @@
</span><span class="cx">
</span><span class="cx"> void WebInspectorClient::drawRect(PageOverlay*, WebCore::GraphicsContext& context, const WebCore::IntRect& /*dirtyRect*/)
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->drawHighlight(context);
</del><ins>+ m_page->corePage()->inspectorController().drawHighlight(context);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool WebInspectorClient::mouseEvent(PageOverlay*, const WebMouseEvent&)
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebCoreSupportWebInspectorFrontendClientcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -40,7 +40,7 @@
</span><span class="cx"> namespace WebKit {
</span><span class="cx">
</span><span class="cx"> WebInspectorFrontendClient::WebInspectorFrontendClient(WebPage* page, WebPage* inspectorPage)
</span><del>- : InspectorFrontendClientLocal(page->corePage()->inspectorController(), inspectorPage->corePage(), adoptPtr(new Settings()))
</del><ins>+ : InspectorFrontendClientLocal(&page->corePage()->inspectorController(), inspectorPage->corePage(), adoptPtr(new Settings()))
</ins><span class="cx"> , m_page(page)
</span><span class="cx"> {
</span><span class="cx"> }
</span><span class="lines">@@ -57,7 +57,7 @@
</span><span class="cx">
</span><span class="cx"> void WebInspectorFrontendClient::closeWindow()
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->disconnectFrontend();
</del><ins>+ m_page->corePage()->inspectorController().disconnectFrontend();
</ins><span class="cx"> m_page->inspector()->didClose();
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebPageWebInspectorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebInspector.cpp (162263 => 162264)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebInspector.cpp        2014-01-18 18:57:59 UTC (rev 162263)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebInspector.cpp        2014-01-18 19:15:33 UTC (rev 162264)
</span><span class="lines">@@ -89,9 +89,9 @@
</span><span class="cx"> m_inspectorPage = WebProcess::shared().webPage(inspectorPageID);
</span><span class="cx"> ASSERT(m_inspectorPage);
</span><span class="cx">
</span><del>- OwnPtr<WebInspectorFrontendClient> frontendClient = adoptPtr(new WebInspectorFrontendClient(m_page, m_inspectorPage));
</del><ins>+ auto frontendClient = std::make_unique<WebInspectorFrontendClient>(m_page, m_inspectorPage);
</ins><span class="cx"> m_frontendClient = frontendClient.get();
</span><del>- m_inspectorPage->corePage()->inspectorController()->setInspectorFrontendClient(frontendClient.release());
</del><ins>+ m_inspectorPage->corePage()->inspectorController().setInspectorFrontendClient(std::move(frontendClient));
</ins><span class="cx"> return m_inspectorPage;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -162,12 +162,12 @@
</span><span class="cx"> // Called by WebInspector messages
</span><span class="cx"> void WebInspector::show()
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::close()
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->close();
</del><ins>+ m_page->corePage()->inspectorController().close();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::didSave(const String& url)
</span><span class="lines">@@ -202,19 +202,19 @@
</span><span class="cx">
</span><span class="cx"> void WebInspector::evaluateScriptForTest(long callID, const String& script)
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->evaluateForTestInFrontend(callID, script);
</del><ins>+ m_page->corePage()->inspectorController().evaluateForTestInFrontend(callID, script);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::showConsole()
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->showConsole();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::showResources()
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->showResources();
</span><span class="cx"> }
</span><span class="lines">@@ -225,7 +225,7 @@
</span><span class="cx"> if (!frame)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->showMainResourceForFrame(frame->coreFrame());
</span><span class="cx"> }
</span><span class="lines">@@ -233,7 +233,7 @@
</span><span class="cx"> void WebInspector::startJavaScriptDebugging()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->setDebuggingEnabled(true);
</span><span class="cx"> #endif
</span><span class="lines">@@ -242,7 +242,7 @@
</span><span class="cx"> void WebInspector::stopJavaScriptDebugging()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->setDebuggingEnabled(false);
</span><span class="cx"> #endif
</span><span class="lines">@@ -251,18 +251,18 @@
</span><span class="cx"> void WebInspector::setJavaScriptProfilingEnabled(bool enabled)
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (!m_frontendClient)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- m_page->corePage()->inspectorController()->setProfilerEnabled(enabled);
</del><ins>+ m_page->corePage()->inspectorController().setProfilerEnabled(enabled);
</ins><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::startJavaScriptProfiling()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->startProfilingJavaScript();
</span><span class="cx"> #endif
</span><span class="lines">@@ -271,7 +271,7 @@
</span><span class="cx"> void WebInspector::stopJavaScriptProfiling()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(JAVASCRIPT_DEBUGGER)
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->stopProfilingJavaScript();
</span><span class="cx"> #endif
</span><span class="lines">@@ -279,14 +279,14 @@
</span><span class="cx">
</span><span class="cx"> void WebInspector::startPageProfiling()
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->setTimelineProfilingEnabled(true);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::stopPageProfiling()
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->show();
</del><ins>+ m_page->corePage()->inspectorController().show();
</ins><span class="cx"> if (m_frontendClient)
</span><span class="cx"> m_frontendClient->setTimelineProfilingEnabled(false);
</span><span class="cx"> }
</span><span class="lines">@@ -310,7 +310,7 @@
</span><span class="cx">
</span><span class="cx"> void WebInspector::dispatchMessageFromRemoteFrontend(const String& message)
</span><span class="cx"> {
</span><del>- m_page->corePage()->inspectorController()->dispatchMessageFromFrontend(message);
</del><ins>+ m_page->corePage()->inspectorController().dispatchMessageFromFrontend(message);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::remoteFrontendConnected()
</span><span class="lines">@@ -319,14 +319,14 @@
</span><span class="cx"> // Switching between in-process and remote inspectors isn't supported yet.
</span><span class="cx"> ASSERT(!m_inspectorPage);
</span><span class="cx">
</span><del>- m_page->corePage()->inspectorController()->connectFrontend(m_frontendChannel);
</del><ins>+ m_page->corePage()->inspectorController().connectFrontend(m_frontendChannel);
</ins><span class="cx"> m_remoteFrontendConnected = true;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebInspector::remoteFrontendDisconnected()
</span><span class="cx"> {
</span><span class="cx"> ASSERT(m_remoteFrontendConnected);
</span><del>- m_page->corePage()->inspectorController()->disconnectFrontend();
</del><ins>+ m_page->corePage()->inspectorController().disconnectFrontend();
</ins><span class="cx"> m_remoteFrontendConnected = false;
</span><span class="cx"> }
</span><span class="cx"> #endif
</span></span></pre>
</div>
</div>
</body>
</html>