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

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 640053007: mac: Add metrics for AppKit Fullscreen style. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Compile failure. Created 6 years, 2 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:
Download patch
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index dd89b52f6e84b34323b6c4b3884c4ed45bea44c8..d864431151bcdaf1189ed7ebe8fc24e74aeee43e 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -21620,6 +21620,11 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="OSX.Fullscreen.Enter" enum="OSXFullscreenParameters">
+ <obsolete>
+ Deprecated as of Chrome 40. See OSX.Fullscreen.Enter.Style,
+ OSX.Fullscreen.Enter.WindowLocation and
+ OSX.Settings.ScreensHaveSeparateSpaces.
+ </obsolete>
<owner>erikchen@chromium.org</owner>
<summary>
This event is recorded each time a user triggers fullscreen for a browser
@@ -21631,6 +21636,34 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="OSX.Fullscreen.Enter.Style" enum="OSXFullscreenStyle">
+ <owner>erikchen@chromium.org</owner>
+ <summary>
+ This event is recorded each time a user triggers fullscreen for a browser
+ window. It indicates the mechanism (immersive vs. AppKit) and the type of
+ AppKit Fullscreen (Presentation Mode vs. Canonical Fullscreen).
+ </summary>
+</histogram>
+
+<histogram name="OSX.Fullscreen.Enter.WindowLocation"
+ enum="OSXFullscreenWindowLocation">
+ <owner>erikchen@chromium.org</owner>
+ <summary>
+ This event is recorded each time a user triggers fullscreen for a browser
+ window. It indicates the screen in which the window was fullscreened, and
+ the number of screens available.
+ </summary>
+</histogram>
+
+<histogram name="OSX.Settings.ScreensHaveSeparateSpaces"
+ enum="OSXScreensHaveSeparateSpaces">
+ <owner>erikchen@chromium.org</owner>
+ <summary>
+ The configuration of the setting &quot;Screens Have Separate Spaces&quot;
+ available in OSX 10.9+.
+ </summary>
+</histogram>
+
<histogram name="OSX.SystemHotkeyMap.LoadSuccess" enum="BooleanSuccess">
<owner>erikchen@chromium.org</owner>
<summary>
@@ -48516,6 +48549,47 @@ To add a new entry, add it with any value and run test to compute valid value.
</int>
</enum>
+<enum name="OSXFullscreenStyle" type="int">
+ <int value="0" label="IMMERSIVE">
+ The window was fullscreened using the immersive mechanism.
+ </int>
+ <int value="1" label="PRESENTATION_MODE">
+ The window was fullscreened using the AppKit mechanism, in Presentation
+ Mode.
+ </int>
+ <int value="2" label="CANONICAL_FULLSCREEN">
+ The window was fullscreened using the AppKit mechanism, in Canonical
+ Fullscreen.
+ </int>
+</enum>
+
+<enum name="OSXFullscreenWindowLocation" type="int">
+ <int value="0" label="PRIMARY_SINGLE_SCREEN">
+ The window was located on the primary screen, and there is only a single
+ screen available.
+ </int>
+ <int value="1" label="PRIMARY_MULTIPLE_SCREEN">
+ The window was located on the primary screen, and there are multiple screens
+ available.
+ </int>
+ <int value="2" label="SECONDARY_MULTIPLE_SCREEN">
+ The window was located on a secondary screen, and there are multiple screens
+ available.
+ </int>
+</enum>
+
+<enum name="OSXScreensHaveSeparateSpaces" type="int">
+ <int value="0" label="CANNOT_HAVE_SEPARATE_SPACES">
+ The &quot;Screens Have Separate Spaces&quot; option is unavailable.
+ </int>
+ <int value="1" label="SEPARATE_SPACES">
+ The &quot;Screens Have Separate Spaces&quot; option is on.
+ </int>
+ <int value="2" label="SHARED_SPACES">
+ The &quot;Screens Have Separate Spaces&quot; option is off.
+ </int>
+</enum>
+
<enum name="OtherPossibleUsernamesUsage" type="int">
<int value="0" label="Nothing to Autofill"/>
<int value="1" label="No other possible usernames"/>
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698