Commit fdce6b60 authored by Regan Hsu's avatar Regan Hsu Committed by Commit Bot

[CrOS Settings] Record settings change metrics initiated by cr network

This CL adds event fires for the custom 'user-action-setting-change'
in os settings ui to record settings metrics for user action internet
page settings change events that may occur outside of settings.

Bug: 1049830
Change-Id: Icb78e07319fc9d81bdf46a78805037e7185bb096
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2063676
Commit-Queue: Regan Hsu <hsuregan@chromium.org>
Reviewed-by: default avatarSteven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743318}
parent 1d647b1b
...@@ -295,10 +295,9 @@ Polymer({ ...@@ -295,10 +295,9 @@ Polymer({
}, },
/** /**
* @param {!CustomEvent<string>} e
* @private * @private
*/ */
onSettingChange_(e) { onSettingChange_() {
settings.recordSettingChange(); settings.recordSettingChange();
}, },
......
...@@ -189,6 +189,7 @@ Polymer({ ...@@ -189,6 +189,7 @@ Polymer({
this.getNetworkConfig_().selectCellularMobileNetwork( this.getNetworkConfig_().selectCellularMobileNetwork(
this.managedProperties.guid, target.value); this.managedProperties.guid, target.value);
this.fire('user-action-setting-change');
}, },
}); });
})(); })();
...@@ -306,6 +306,7 @@ Polymer({ ...@@ -306,6 +306,7 @@ Polymer({
this.delayUpdateLockEnabled_(); this.delayUpdateLockEnabled_();
} }
}); });
this.fire('user-action-setting-change');
}, },
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment