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

Unified Diff: chrome/renderer/resources/extension_process_bindings.js

Issue 454019: Addition of optional giveFocus parameter to experimental.popup.show(...) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years 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 side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/resources/extension_process_bindings.js
===================================================================
--- chrome/renderer/resources/extension_process_bindings.js (revision 33889)
+++ chrome/renderer/resources/extension_process_bindings.js (working copy)
@@ -425,19 +425,31 @@
this.definition.parameters[0],
{
type: "object",
- name: "domAnchor",
+ name: "showDetails",
properties: {
- top: { type: "integer", minimum: 0 },
- left: { type: "integer", minimum: 0 },
- width: { type: "integer", minimum: 0 },
- height: { type: "integer", minimum: 0 }
+ domAnchor: {
+ type: "object",
+ properties: {
+ top: { type: "integer", minimum: 0 },
+ left: { type: "integer", minimum: 0 },
+ width: { type: "integer", minimum: 0 },
+ height: { type: "integer", minimum: 0 }
+ }
+ },
+ giveFocus: {
+ type: "boolean",
+ optional: true
+ }
}
},
this.definition.parameters[2]
];
return sendRequest(this.name,
[url,
- getAbsoluteRect(showDetails.relativeTo),
+ {
+ domAnchor: getAbsoluteRect(showDetails.relativeTo),
+ giveFocus: showDetails.giveFocus
+ },
callback],
internalSchema);
}

Powered by Google App Engine
This is Rietveld 408576698