chrome/common/extensions/api/debugger.json - Issue 12304021: Allow chrome.debugger API to attach to extension background pages.

Side by Side Diff: chrome/common/extensions/api/debugger.json

Issue 12304021: Allow chrome.debugger API to attach to extension background pages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Updated copyright Created 7 years, 10 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« chrome/browser/extensions/api/debugger/debugger_api.cc ('K') | « chrome/common/chrome_switches.cc ('k') | chrome/test/data/extensions/api_test/debugger/background.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.	1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.

2 // Use of this source code is governed by a BSD-style license that can be	2 // Use of this source code is governed by a BSD-style license that can be

3 // found in the LICENSE file.	3 // found in the LICENSE file.

4	4

5 [	5 [

6 {	6 {

7 "namespace": "debugger",	7 "namespace": "debugger",

8 "types": [	8 "types": [

9 {	9 {

10 "id": "Debuggee",	10 "id": "Debuggee",

11 "type": "object",	11 "type": "object",

12 "description": "Debuggee identifier.",	12 "description": "Debuggee identifier. Either tabId or extensionId must be specified",

13 "properties": {	13 "properties": {

14 "tabId": { "type": "integer", "description": "The id of the tab which you intend to debug." }	14 "tabId": { "type": "integer", "optional": true, "description": "The id of the tab which you intend to debug." },

	15 "extensionId": { "type": "string", "optional": true, "description": "T he id of the extension which you intend to debug." }
	Matt Perry 2013/02/20 19:52:01 add a "nodoc": true field to this - since you need add a "nodoc": true field to this - since you need a command line flag, it seems like this version has very limited use. Vladislav Kaznacheev 2013/02/21 09:39:09 This API is already a sort of a "specialist" one u This API is already a sort of a "specialist" one used by developer tools authors (I am aware of exactly one external user). I think having this property documented is not likely to confuse them more than they already are :) On 2013/02/20 19:52:01, Matt Perry wrote: Show quoted text > add a "nodoc": true field to this - since you need a command line flag, it seems > like this version has very limited use. Matt Perry 2013/02/21 18:35:33 OK, could you update the description to note that Show quoted text On 2013/02/21 09:39:09, Vladislav Kaznacheev wrote: > This API is already a sort of a "specialist" one used by developer tools authors > (I am aware of exactly one external user). I think having this property > documented is not likely to confuse them more than they already are :) > > On 2013/02/20 19:52:01, Matt Perry wrote: > > add a "nodoc": true field to this - since you need a command line flag, it > seems > > like this version has very limited use. > OK, could you update the description to note that it won't work without the flag? Vladislav Kaznacheev 2013/02/26 10:55:38 Added a flag to chrome:flags and mentioned it in t Added a flag to chrome:flags and mentioned it in the "extensionID" property description. On 2013/02/21 18:35:33, Matt Perry wrote: Show quoted text > On 2013/02/21 09:39:09, Vladislav Kaznacheev wrote: > > This API is already a sort of a "specialist" one used by developer tools > authors > > (I am aware of exactly one external user). I think having this property > > documented is not likely to confuse them more than they already are :) > > > > On 2013/02/20 19:52:01, Matt Perry wrote: > > > add a "nodoc": true field to this - since you need a command line flag, it > > seems > > > like this version has very limited use. > > > > OK, could you update the description to note that it won't work without the > flag?
15 }	16 }

16 }	17 }

17 ],	18 ],

18 "functions": [	19 "functions": [

19 {	20 {

20 "name": "attach",	21 "name": "attach",

21 "type": "function",	22 "type": "function",

22 "description": "Attaches debugger to the given target.",	23 "description": "Attaches debugger to the given target.",

23 "parameters": [	24 "parameters": [

24 {	25 {

(...skipping 113 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
138 "type": "string",	139 "type": "string",

139 "name": "reason",	140 "name": "reason",

140 "description": "Connection termination reason.",	141 "description": "Connection termination reason.",

141 "enum": [ "target_closed", "canceled_by_user", "replaced_with_devtoo ls" ]	142 "enum": [ "target_closed", "canceled_by_user", "replaced_with_devtoo ls" ]

142 }	143 }

143 ]	144 ]

144 }	145 }

145 ]	146 ]

146 }	147 }

147 ]	148 ]

OLD	NEW