New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 687710 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

"specifying" misspelled twice in AFE source code

Reported by jrbarnette@chromium.org, Feb 1 2017

Issue description

In the autotest sources:
$ find frontend -name '*.java' | xargs grep specifing
frontend/client/src/autotest/afe/HostSelectorDisplay.java:        tabPanel.add(hostnamePanel, "By specifing hostnames");
frontend/client/src/autotest/afe/HostSelectorDisplay.java:        tabPanel.add(labelPanel, "By specifing host labels");

This misspelling shows up as tab names on the "Create Job"
page of the AFE web UI.
 
Owner: jrbarnette@chromium.org
Status: Started (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/dfb4e30592ddeaceb270df3fbab3ff3faefa5396

commit dfb4e30592ddeaceb270df3fbab3ff3faefa5396
Author: Richard Barnette <jrbarnette@chromium.org>
Date: Fri Feb 03 20:37:41 2017

[autotest] Correctly spell "specifying" in the AFE GWT code.

BUG= chromium:687710 
TEST=Look at the AFE in a local Autotest instance

Change-Id: I3137bc6464c8b6012191e698f13139b384e0a083
Reviewed-on: https://chromium-review.googlesource.com/435892
Commit-Ready: Richard Barnette <jrbarnette@chromium.org>
Tested-by: Richard Barnette <jrbarnette@chromium.org>
Reviewed-by: Shuqian Zhao <shuqianz@chromium.org>

[modify] https://crrev.com/dfb4e30592ddeaceb270df3fbab3ff3faefa5396/frontend/client/src/autotest/afe/HostSelectorDisplay.java

Labels: cros-infra-fixedit-q117
Status: Fixed (was: Started)

Comment 4 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 5 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 6 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 7 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment