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

Unified Diff: third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js

Issue 2353033004: [DevTools] Expose left and right toolbar in TabbedPane. (Closed)
Patch Set: review comment Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js b/third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js
index 584a011a526f0ed11a0479a81ee6aa231761b196..94a5c81b832727fb6a92c82f297dc23a4f9885e2 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js
@@ -173,6 +173,55 @@ WebInspector.Toolbar.prototype = {
lastSeparator.setVisible(false);
this.element.classList.toggle("hidden", !!lastSeparator && lastSeparator.visible() && !nonSeparatorVisible);
+ },
+
+ /**
+ * @param {string} location
+ */
+ appendLocationItems: function(location)
+ {
+ var extensions = self.runtime.extensions(WebInspector.ToolbarItem.Provider);
+ var promises = [];
+ for (var i = 0; i < extensions.length; ++i) {
+ if (extensions[i].descriptor()["location"] === location)
+ promises.push(resolveItem(extensions[i]));
+ }
+ Promise.all(promises).then(appendItemsInOrder.bind(this));
+
+ /**
+ * @param {!Runtime.Extension} extension
+ * @return {!Promise.<?WebInspector.ToolbarItem>}
+ */
+ function resolveItem(extension)
+ {
+ var descriptor = extension.descriptor();
+ if (descriptor["separator"])
+ return Promise.resolve(/** @type {?WebInspector.ToolbarItem} */(new WebInspector.ToolbarSeparator()));
+ if (descriptor["actionId"])
+ return Promise.resolve(WebInspector.Toolbar.createActionButtonForId(descriptor["actionId"]));
+ return extension.instance().then(fetchItemFromProvider);
+
+ /**
+ * @param {!Object} provider
+ */
+ function fetchItemFromProvider(provider)
+ {
+ return /** @type {!WebInspector.ToolbarItem.Provider} */ (provider).item();
+ }
+ }
+
+ /**
+ * @param {!Array.<?WebInspector.ToolbarItem>} items
+ * @this {WebInspector.Toolbar}
+ */
+ function appendItemsInOrder(items)
+ {
+ for (var i = 0; i < items.length; ++i) {
+ var item = items[i];
+ if (item)
+ this.appendToolbarItem(item);
+ }
+ }
}
}
@@ -945,67 +994,3 @@ WebInspector.ToolbarCheckbox.prototype = {
__proto__: WebInspector.ToolbarItem.prototype
}
-/**
- * @constructor
- * @extends {WebInspector.Toolbar}
- * @param {string} location
- * @param {!Element=} parentElement
- */
-WebInspector.ExtensibleToolbar = function(location, parentElement)
-{
- WebInspector.Toolbar.call(this, "", parentElement);
- this._loadItems(location);
-}
-
-WebInspector.ExtensibleToolbar.prototype = {
- /**
- * @param {string} location
- */
- _loadItems: function(location)
- {
- var extensions = self.runtime.extensions(WebInspector.ToolbarItem.Provider);
- var promises = [];
- for (var i = 0; i < extensions.length; ++i) {
- if (extensions[i].descriptor()["location"] === location)
- promises.push(resolveItem(extensions[i]));
- }
- Promise.all(promises).then(appendItemsInOrder.bind(this));
-
- /**
- * @param {!Runtime.Extension} extension
- * @return {!Promise.<?WebInspector.ToolbarItem>}
- */
- function resolveItem(extension)
- {
- var descriptor = extension.descriptor();
- if (descriptor["separator"])
- return Promise.resolve(/** @type {?WebInspector.ToolbarItem} */(new WebInspector.ToolbarSeparator()));
- if (descriptor["actionId"])
- return Promise.resolve(WebInspector.Toolbar.createActionButtonForId(descriptor["actionId"]));
- return extension.instance().then(fetchItemFromProvider);
-
- /**
- * @param {!Object} provider
- */
- function fetchItemFromProvider(provider)
- {
- return /** @type {!WebInspector.ToolbarItem.Provider} */ (provider).item();
- }
- }
-
- /**
- * @param {!Array.<?WebInspector.ToolbarItem>} items
- * @this {WebInspector.ExtensibleToolbar}
- */
- function appendItemsInOrder(items)
- {
- for (var i = 0; i < items.length; ++i) {
- var item = items[i];
- if (item)
- this.appendToolbarItem(item);
- }
- }
- },
-
- __proto__: WebInspector.Toolbar.prototype
-}
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/ui/TabbedPane.js ('k') | third_party/WebKit/Source/devtools/front_end/ui/View.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698