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

Issue 699287 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Email drop down is not needed in Guest/Supervised user session feedback window

Project Member Reported by sdantul...@chromium.org, Mar 7 2017

Issue description

Google Chrome	58.0.3029.6 (Official Build) dev (64-bit)
Revision	0
Platform	9334.2.0 (Official Build) dev-channel peppy

What steps will reproduce the problem?
1. Login as guest user or supervised user
2. Alt + Shift + i to open feedback window
3. Check drop down beside Email field

What is the expected result?
Drop down is not needed as there is only one option available

What happens instead?
Drop down is shown.

Attached screenshot.
 
Labels: -Pri-3 M-58 OS-Chrome Pri-2
Summary: Email drop down is not needed in Guest/Supervised user session feedback window (was: Guest/Supervised user session)
Attached screenshot.
FeedbackUI.png
55.1 KB View Download
Owner: afakhry@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/449ca301d5722d443b57ab5fd5db36a57b9f821a

commit 449ca301d5722d443b57ab5fd5db36a57b9f821a
Author: afakhry <afakhry@chromium.org>
Date: Sat Mar 11 00:09:51 2017

Prevent adding an empty email option in the feedback app in guest mode

BUG= 699287 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2742103002
Cr-Commit-Position: refs/heads/master@{#456223}

[modify] https://crrev.com/449ca301d5722d443b57ab5fd5db36a57b9f821a/chrome/browser/resources/feedback/js/feedback.js

Labels: Merge-Request-58
Project Member

Comment 5 by sheriffbot@chromium.org, Mar 12 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 13 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/644edabf28d40898cfc8180c6ec8b7933b7ab2e4

commit 644edabf28d40898cfc8180c6ec8b7933b7ab2e4
Author: Ahmed Fakhry <afakhry@google.com>
Date: Mon Mar 13 17:00:24 2017

[Merge to M58] Prevent adding an empty email option in the feedback app in guest mode

TBR=xiyuan@chromium.org
BUG= 699287 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2742103002
Cr-Commit-Position: refs/heads/master@{#456223}
(cherry picked from commit 449ca301d5722d443b57ab5fd5db36a57b9f821a)

Review-Url: https://codereview.chromium.org/2747813002 .
Cr-Commit-Position: refs/branch-heads/3029@{#153}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/644edabf28d40898cfc8180c6ec8b7933b7ab2e4/chrome/browser/resources/feedback/js/feedback.js

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
9334.11.0 / 58.0.3029.19

Sign in to add a comment