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

Unified Diff: chrome/common/extensions/api/windows.json

Issue 2779683004: Send UMA about requested window state from tabs API (Closed)
Patch Set: mpearson's Created 3 years, 9 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
« no previous file with comments | « chrome/browser/extensions/api/tabs/tabs_api.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/api/windows.json
diff --git a/chrome/common/extensions/api/windows.json b/chrome/common/extensions/api/windows.json
index 17eb6fd896ae57299796cf2542f38149dccbdf53..0e8af54caecd527552ece2be989daf2c937948b9 100644
--- a/chrome/common/extensions/api/windows.json
+++ b/chrome/common/extensions/api/windows.json
@@ -20,6 +20,11 @@
"id": "WindowState",
"type": "string",
"description": "The state of this browser window. Under some circumstances a Window may not be assigned state property, for example when querying closed windows from the $(ref:sessions) API.",
+ // WARNING: These values are written to logs. New enum values can be
+ // added, but existing enums must never be renumbered or deleted and
+ // reused. If something needs to be removed, make sure to remove the
+ // "TabsApi.RequestedWindowState" histogram as it will be no longer
+ // valid, and make sure to mark it as obsolete in histograms.xml.
"enum": [{
"name": "normal",
"description": "Normal window state (i.e. not minimized, maximized, or fullscreen)."
« no previous file with comments | « chrome/browser/extensions/api/tabs/tabs_api.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698