[Devtools] Add capability for the log domain

BUG=594160

Review-Url: https://codereview.chromium.org/2345873002
Cr-Commit-Position: refs/heads/master@{#419304}

third_party/WebKit/LayoutTests/http/tests/inspector/workspace-test.js

@@ -34,10 +34,14 @@ InspectorTest.createWorkspace = function(ignoreEvents)
 }
 
 InspectorTest._mockTargetId = 1;
+InspectorTest._pageCapabilities =
+    WebInspector.Target.Capability.Browser | WebInspector.Target.Capability.DOM |
+    WebInspector.Target.Capability.JS | WebInspector.Target.Capability.Log |
+    WebInspector.Target.Capability.Network | WebInspector.Target.Capability.Worker;
 
 InspectorTest.createMockTarget = function(id, debuggerModelConstructor, capabilities)
 {
-    capabilities = capabilities || (WebInspector.Target.Capability.Browser | WebInspector.Target.Capability.JS | WebInspector.Target.Capability.Network | WebInspector.Target.Capability.Worker);
+    capabilities = capabilities || InspectorTest._pageCapabilities;
     var MockTarget = function(name, connection, callback)
     {
         WebInspector.Target.call(this, InspectorTest.testTargetManager, name, capabilities, connection, null, callback);

third_party/WebKit/Source/devtools/front_end/main/Main.js

@@ -290,9 +290,14 @@ WebInspector.Main.prototype = {
             WebInspector.RemoteDebuggingTerminatedScreen.show(event.data.reason);
         }
 
-        var capabilities = WebInspector.Target.Capability.Browser | WebInspector.Target.Capability.JS | WebInspector.Target.Capability.Network | WebInspector.Target.Capability.Worker | WebInspector.Target.Capability.DOM;
+        var capabilities =
+            WebInspector.Target.Capability.Browser | WebInspector.Target.Capability.DOM |
+            WebInspector.Target.Capability.JS | WebInspector.Target.Capability.Log |
+            WebInspector.Target.Capability.Network | WebInspector.Target.Capability.Worker;
         if (Runtime.queryParam("isSharedWorker"))
-            capabilities = WebInspector.Target.Capability.Browser | WebInspector.Target.Capability.Network | WebInspector.Target.Capability.Worker;
+            capabilities =
+                WebInspector.Target.Capability.Browser | WebInspector.Target.Capability.Log |
+                WebInspector.Target.Capability.Network | WebInspector.Target.Capability.Worker;
         else if (Runtime.queryParam("v8only"))
             capabilities = WebInspector.Target.Capability.JS;
 

third_party/WebKit/Source/devtools/front_end/sdk/ConsoleModel.js

@@ -32,8 +32,9 @@
  * @constructor
  * @extends {WebInspector.SDKModel}
  * @param {!WebInspector.Target} target
+ * @param {?Protocol.LogAgent} logAgent
  */
-WebInspector.ConsoleModel = function(target)
+WebInspector.ConsoleModel = function(target, logAgent)
 {
     WebInspector.SDKModel.call(this, WebInspector.ConsoleModel, target);
 
@@ -44,9 +45,11 @@ WebInspector.ConsoleModel = function(target)
     this._warnings = 0;
     this._errors = 0;
     this._revokedErrors = 0;
-    this._logAgent = target.logAgent();
-    target.registerLogDispatcher(new WebInspector.LogDispatcher(this));
-    this._logAgent.enable();
+    this._logAgent = logAgent;
+    if (this._logAgent) {
+        target.registerLogDispatcher(new WebInspector.LogDispatcher(this));
+        this._logAgent.enable();
+    }
 }
 
 /** @enum {symbol} */
@@ -134,7 +137,7 @@ WebInspector.ConsoleModel.prototype = {
 
     requestClearMessages: function()
     {
-        this._logAgent.clear();
+        this._logAgent && this._logAgent.clear();
         this.clear();
     },
 

third_party/WebKit/Source/devtools/front_end/sdk/ServiceWorkerManager.js

@@ -436,7 +436,9 @@ WebInspector.ServiceWorker = function(manager, workerId, url, versionId)
     this._scope = parsedURL ? parsedURL.host + parsedURL.folderPathComponents : "";
 
     this._manager._workers.set(workerId, this);
-    var capabilities = WebInspector.Target.Capability.Network | WebInspector.Target.Capability.Worker;
+    var capabilities =
+        WebInspector.Target.Capability.Log | WebInspector.Target.Capability.Network |
+        WebInspector.Target.Capability.Worker;
     this._target = WebInspector.targetManager.createTarget(this._name, capabilities, this._connection, manager.target());
     this._target[WebInspector.ServiceWorker.Symbol] = this;
     this._manager.dispatchEventToListeners(WebInspector.ServiceWorkerManager.Events.WorkersUpdated);

third_party/WebKit/Source/devtools/front_end/sdk/Target.js

@@ -34,11 +34,12 @@ WebInspector.Target = function(targetManager, name, capabilitiesMask, connection
  */
 WebInspector.Target.Capability = {
     Browser: 1,
-    JS: 2,
-    Network: 4,
-    Worker: 8,
-    DOM: 16
-}
+    DOM: 2,
+    JS: 4,
+    Log: 8,
+    Network: 16,
+    Worker: 32
+};
 
 WebInspector.Target._nextId = 1;
 
@@ -120,6 +121,14 @@ WebInspector.Target.prototype = {
         return this.hasAllCapabilities(WebInspector.Target.Capability.JS);
     },
 
+    /**
+     * @return {boolean}
+     */
+    hasLogCapability: function()
+    {
+        return this.hasAllCapabilities(WebInspector.Target.Capability.Log);
+    },
+
     /**
      * @return {boolean}
      */

third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js

@@ -190,8 +190,10 @@ WebInspector.TargetManager.prototype = {
     {
         var target = new WebInspector.Target(this, name, capabilitiesMask, connection, parentTarget);
 
+        var logAgent = target.hasLogCapability() ? target.logAgent() : null;
+
         /** @type {!WebInspector.ConsoleModel} */
-        target.consoleModel = new WebInspector.ConsoleModel(target);
+        target.consoleModel = new WebInspector.ConsoleModel(target, logAgent);
         /** @type {!WebInspector.RuntimeModel} */
         target.runtimeModel = new WebInspector.RuntimeModel(target);
 

third_party/WebKit/Source/devtools/front_end/sdk/WorkerManager.js

@@ -101,7 +101,7 @@ WebInspector.WorkerManager.prototype = {
 
         var parsedURL = url.asParsedURL();
         var workerName = parsedURL ? parsedURL.lastPathComponentWithFragment() : "#" + (++this._lastAnonymousTargetId);
-        var target = WebInspector.targetManager.createTarget(workerName, WebInspector.Target.Capability.JS, connection, this.target());
+        var target = WebInspector.targetManager.createTarget(workerName, WebInspector.Target.Capability.JS | WebInspector.Target.Capability.Log, connection, this.target());
         this._targetsByWorkerId.set(workerId, target);
 
         // Only pause new worker if debugging SW - we are going through the