Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(479)

Side by Side Diff: third_party/WebKit/Source/core/inspector/browser_protocol.json

Issue 2226323002: Resize DevTools target frames. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use MAYBE_ pattern to disable test on Android. Created 4 years, 4 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
« no previous file with comments | « headless/lib/headless_devtools_client_browsertest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 { 1 {
2 "version": { "major": "1", "minor": "1" }, 2 "version": { "major": "1", "minor": "1" },
3 "domains": [{ 3 "domains": [{
4 "domain": "Inspector", 4 "domain": "Inspector",
5 "experimental": true, 5 "experimental": true,
6 "types": [], 6 "types": [],
7 "commands": [ 7 "commands": [
8 { 8 {
9 "name": "enable", 9 "name": "enable",
10 "description": "Enables inspector domain notifications.", 10 "description": "Enables inspector domain notifications.",
(...skipping 740 matching lines...) Expand 10 before | Expand all | Expand 10 after
751 "description": "Requests that page scale factor is reset to init ial values." 751 "description": "Requests that page scale factor is reset to init ial values."
752 }, 752 },
753 { 753 {
754 "name": "setPageScaleFactor", 754 "name": "setPageScaleFactor",
755 "description": "Sets a specified page scale factor.", 755 "description": "Sets a specified page scale factor.",
756 "parameters": [ 756 "parameters": [
757 { "name": "pageScaleFactor", "type": "number", "description" : "Page scale factor." } 757 { "name": "pageScaleFactor", "type": "number", "description" : "Page scale factor." }
758 ] 758 ]
759 }, 759 },
760 { 760 {
761 "name": "getFrameSize",
dgozman 2016/08/18 20:16:25 As you discussed with Pavel, let's remove this and
762 "description": "Returns the size of the target's (non-overridden ) frame.",
763 "returns": [
764 { "name": "width", "type": "integer", "description": "Frame width (DIP)." },
765 { "name": "height", "type": "integer", "description": "Frame height (DIP)." }
766 ],
767 "handlers": ["browser"]
768 },
769 {
770 "name": "setFrameSize",
dgozman 2016/08/18 20:16:25 I'm worried this name is confusing as a part of pr
Eric Seckler 2016/08/18 22:04:40 I'd find it clearer if the command's name actually
dgozman 2016/08/19 17:52:29 I personally like setVisibleSize more, because it
771 "description": "Resize the target's frame. Note that this does n ot affect the frame's container (e.g. browser window). Not supported on Android. ",
772 "parameters": [
773 { "name": "width", "type": "integer", "description": "Frame width (DIP)." },
774 { "name": "height", "type": "integer", "description": "Frame height (DIP)." }
775 ],
776 "handlers": ["browser"]
777 },
778 {
761 "name": "setScriptExecutionDisabled", 779 "name": "setScriptExecutionDisabled",
762 "description": "Switches script execution in the page.", 780 "description": "Switches script execution in the page.",
763 "parameters": [ 781 "parameters": [
764 { "name": "value", "type": "boolean", "description": "Whethe r script execution should be disabled in the page." } 782 { "name": "value", "type": "boolean", "description": "Whethe r script execution should be disabled in the page." }
765 ] 783 ]
766 }, 784 },
767 { 785 {
768 "name": "setGeolocationOverride", 786 "name": "setGeolocationOverride",
769 "description": "Overrides the Geolocation Position or Error. Omi tting any of the parameters emulates position unavailable.", 787 "description": "Overrides the Geolocation Position or Error. Omi tting any of the parameters emulates position unavailable.",
770 "parameters": [ 788 "parameters": [
(...skipping 3685 matching lines...) Expand 10 before | Expand all | Expand 10 after
4456 "description": "Informs that port was successfully bound and got a specified connection id.", 4474 "description": "Informs that port was successfully bound and got a specified connection id.",
4457 "parameters": [ 4475 "parameters": [
4458 {"name": "port", "type": "integer", "description": "Port num ber that was successfully bound." }, 4476 {"name": "port", "type": "integer", "description": "Port num ber that was successfully bound." },
4459 {"name": "connectionId", "type": "string", "description": "C onnection id to be used." } 4477 {"name": "connectionId", "type": "string", "description": "C onnection id to be used." }
4460 ], 4478 ],
4461 "handlers": ["browser"] 4479 "handlers": ["browser"]
4462 } 4480 }
4463 ] 4481 ]
4464 }] 4482 }]
4465 } 4483 }
OLDNEW
« no previous file with comments | « headless/lib/headless_devtools_client_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698