Commit 8fdde830 authored by David Tseng's avatar David Tseng Committed by Chromium LUCI CQ

Remove @public usage in c/b/r/c/accessibility

https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler#public
"
@public
Indicates that a member or property is public. A property marked @public
is accessible to all code in the any file. This is the implicit default
and rarely used. This is not used to indicate that name should be
preserved in obfuscating builds see @export.
"

AX-Relnotes: n/a
Change-Id: I9939c883184b4ec163033d36813e9687fdf7bc05
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2582178
Commit-Queue: David Tseng <dtseng@chromium.org>
Reviewed-by: default avatarAkihiro Ota <akihiroota@chromium.org>
Cr-Commit-Position: refs/heads/master@{#835389}
parent 7b0aafc1
...@@ -150,7 +150,6 @@ class InputHandler { ...@@ -150,7 +150,6 @@ class InputHandler {
* forwarded to us from the SelectToSpeakEventHandler so they should * forwarded to us from the SelectToSpeakEventHandler so they should
* be interpreted as global events on the whole screen, not local to * be interpreted as global events on the whole screen, not local to
* any particular window. * any particular window.
* @public
*/ */
setUpEventListeners() { setUpEventListeners() {
document.addEventListener('keydown', this.onKeyDown_.bind(this)); document.addEventListener('keydown', this.onKeyDown_.bind(this));
...@@ -168,7 +167,6 @@ class InputHandler { ...@@ -168,7 +167,6 @@ class InputHandler {
* Change whether or not we are tracking the mouse. * Change whether or not we are tracking the mouse.
* @param {boolean} tracking True if we should start tracking the mouse, false * @param {boolean} tracking True if we should start tracking the mouse, false
* otherwise. * otherwise.
* @public
*/ */
setTrackingMouse(tracking) { setTrackingMouse(tracking) {
this.trackingMouse_ = tracking; this.trackingMouse_ = tracking;
...@@ -176,7 +174,6 @@ class InputHandler { ...@@ -176,7 +174,6 @@ class InputHandler {
/** /**
* Gets the rect that has been drawn by clicking and dragging the mouse. * Gets the rect that has been drawn by clicking and dragging the mouse.
* @public
*/ */
getMouseRect() { getMouseRect() {
return RectUtil.rectFromPoints( return RectUtil.rectFromPoints(
...@@ -186,7 +183,6 @@ class InputHandler { ...@@ -186,7 +183,6 @@ class InputHandler {
/** /**
* Sets the date at which we last wanted the clipboard data to be read. * Sets the date at which we last wanted the clipboard data to be read.
* @public
*/ */
onRequestReadClipboardData() { onRequestReadClipboardData() {
this.lastReadClipboardDataTime_ = new Date(); this.lastReadClipboardDataTime_ = new Date();
...@@ -202,7 +198,6 @@ class InputHandler { ...@@ -202,7 +198,6 @@ class InputHandler {
* @return {boolean} True if the default action should be performed; * @return {boolean} True if the default action should be performed;
* we always return false because we don't want any other event * we always return false because we don't want any other event
* handlers to run. * handlers to run.
* @public
*/ */
onMouseDown_(evt) { onMouseDown_(evt) {
// If the user hasn't clicked 'search', or if they are currently // If the user hasn't clicked 'search', or if they are currently
...@@ -231,7 +226,6 @@ class InputHandler { ...@@ -231,7 +226,6 @@ class InputHandler {
* *
* @param {!Event} evt * @param {!Event} evt
* @return {boolean} True if the default action should be performed. * @return {boolean} True if the default action should be performed.
* @public
*/ */
onMouseUp_(evt) { onMouseUp_(evt) {
if (!this.trackingMouse_) { if (!this.trackingMouse_) {
...@@ -258,7 +252,6 @@ class InputHandler { ...@@ -258,7 +252,6 @@ class InputHandler {
/** /**
* Visible for testing. * Visible for testing.
* @param {!Event} evt * @param {!Event} evt
* @public
*/ */
onKeyDown_(evt) { onKeyDown_(evt) {
this.keysCurrentlyDown_.add(evt.keyCode); this.keysCurrentlyDown_.add(evt.keyCode);
...@@ -281,7 +274,6 @@ class InputHandler { ...@@ -281,7 +274,6 @@ class InputHandler {
/** /**
* Visible for testing. * Visible for testing.
* @param {!Event} evt * @param {!Event} evt
* @public
*/ */
onKeyUp_(evt) { onKeyUp_(evt) {
if (evt.keyCode === SelectToSpeak.READ_SELECTION_KEY_CODE) { if (evt.keyCode === SelectToSpeak.READ_SELECTION_KEY_CODE) {
......
...@@ -10,7 +10,6 @@ class MetricsUtils { ...@@ -10,7 +10,6 @@ class MetricsUtils {
/** /**
* Records a cancel event if speech was in progress. * Records a cancel event if speech was in progress.
* @param {boolean} speaking Whether speech was in progress * @param {boolean} speaking Whether speech was in progress
* @public
*/ */
static recordCancelIfSpeaking(speaking) { static recordCancelIfSpeaking(speaking) {
if (speaking) { if (speaking) {
...@@ -24,7 +23,6 @@ class MetricsUtils { ...@@ -24,7 +23,6 @@ class MetricsUtils {
* that reflects how this event was triggered by the user. * that reflects how this event was triggered by the user.
* @param {PrefsManager} prefsManager A PrefsManager with the users's current * @param {PrefsManager} prefsManager A PrefsManager with the users's current
* preferences. * preferences.
* @public
*/ */
static recordStartEvent(method, prefsManager) { static recordStartEvent(method, prefsManager) {
chrome.metricsPrivate.recordUserAction(MetricsUtils.START_SPEECH_METRIC); chrome.metricsPrivate.recordUserAction(MetricsUtils.START_SPEECH_METRIC);
...@@ -50,7 +48,6 @@ class MetricsUtils { ...@@ -50,7 +48,6 @@ class MetricsUtils {
/** /**
* Records a user-requested state change event from a given state. * Records a user-requested state change event from a given state.
* @param {number} changeType * @param {number} changeType
* @public
*/ */
static recordSelectToSpeakStateChangeEvent(changeType) { static recordSelectToSpeakStateChangeEvent(changeType) {
chrome.metricsPrivate.recordEnumerationValue( chrome.metricsPrivate.recordEnumerationValue(
......
...@@ -217,7 +217,6 @@ class PrefsManager { ...@@ -217,7 +217,6 @@ class PrefsManager {
* Loads preferences from chrome.storage, sets default values if * Loads preferences from chrome.storage, sets default values if
* necessary, and registers a listener to update prefs when they * necessary, and registers a listener to update prefs when they
* change. * change.
* @public
*/ */
initPreferences() { initPreferences() {
var updatePrefs = () => { var updatePrefs = () => {
...@@ -273,7 +272,6 @@ class PrefsManager { ...@@ -273,7 +272,6 @@ class PrefsManager {
* Generates the basic speech options for Select-to-Speak based on user * Generates the basic speech options for Select-to-Speak based on user
* preferences. Call for each chrome.tts.speak. * preferences. Call for each chrome.tts.speak.
* @return {!TtsOptions} options The TTS options. * @return {!TtsOptions} options The TTS options.
* @public
*/ */
speechOptions() { speechOptions() {
const options = {enqueue: true}; const options = {enqueue: true};
...@@ -309,7 +307,6 @@ class PrefsManager { ...@@ -309,7 +307,6 @@ class PrefsManager {
/** /**
* Gets the user's word highlighting enabled preference. * Gets the user's word highlighting enabled preference.
* @return {boolean} True if word highlighting is enabled. * @return {boolean} True if word highlighting is enabled.
* @public
*/ */
wordHighlightingEnabled() { wordHighlightingEnabled() {
return this.wordHighlight_; return this.wordHighlight_;
...@@ -318,7 +315,6 @@ class PrefsManager { ...@@ -318,7 +315,6 @@ class PrefsManager {
/** /**
* Gets the user's word highlighting color preference. * Gets the user's word highlighting color preference.
* @return {string} Highlight color. * @return {string} Highlight color.
* @public
*/ */
highlightColor() { highlightColor() {
return this.highlightColor_; return this.highlightColor_;
...@@ -328,7 +324,6 @@ class PrefsManager { ...@@ -328,7 +324,6 @@ class PrefsManager {
* Gets the focus ring color. This is not currently a user preference but it * Gets the focus ring color. This is not currently a user preference but it
* could be in the future; stored here for similarity to highlight color. * could be in the future; stored here for similarity to highlight color.
* @return {string} Highlight color. * @return {string} Highlight color.
* @public
*/ */
focusRingColor() { focusRingColor() {
return this.color_; return this.color_;
...@@ -338,7 +333,6 @@ class PrefsManager { ...@@ -338,7 +333,6 @@ class PrefsManager {
* Gets the user's focus ring background color. If the user disabled greying * Gets the user's focus ring background color. If the user disabled greying
* out the background, alpha will be set to fully transparent. * out the background, alpha will be set to fully transparent.
* @return {boolean} True if the background shade should be drawn. * @return {boolean} True if the background shade should be drawn.
* @public
*/ */
backgroundShadingEnabled() { backgroundShadingEnabled() {
return this.backgroundShadingEnabled_; return this.backgroundShadingEnabled_;
...@@ -349,7 +343,6 @@ class PrefsManager { ...@@ -349,7 +343,6 @@ class PrefsManager {
* to navigate to next/previous sentences, paragraphs, and more. * to navigate to next/previous sentences, paragraphs, and more.
* @return {boolean} True if navigation controls should be shown when STS is * @return {boolean} True if navigation controls should be shown when STS is
* active. * active.
* @public
*/ */
navigationControlsEnabled() { navigationControlsEnabled() {
return this.navigationControlsEnabled_; return this.navigationControlsEnabled_;
...@@ -359,7 +352,6 @@ class PrefsManager { ...@@ -359,7 +352,6 @@ class PrefsManager {
/** /**
* Constant representing the system TTS voice. * Constant representing the system TTS voice.
* @type {string} * @type {string}
* @public
*/ */
PrefsManager.SYSTEM_VOICE = 'select_to_speak_system_voice'; PrefsManager.SYSTEM_VOICE = 'select_to_speak_system_voice';
......
...@@ -9,9 +9,9 @@ class FocusData { ...@@ -9,9 +9,9 @@ class FocusData {
* @param {!SAChildNode} focus Must be a child of |group|. * @param {!SAChildNode} focus Must be a child of |group|.
*/ */
constructor(group, focus) { constructor(group, focus) {
/** @public {!SARootNode} */ /** @type {!SARootNode} */
this.group = group; this.group = group;
/** @public {!SAChildNode} */ /** @type {!SAChildNode} */
this.focus = focus; this.focus = focus;
} }
......
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