Feedback: User email section should only be visible when useful

Feedback: User email section should only be visible when useful

When user is not logged on, default is to report anonymously. No
user email is available for us. In this case, there's no need to
show the user email section. So we'll keep it hidden until a valid
email entry is added.

BUG=711357
TEST=login in guest mode, open feedback app (Alt+Shift+i), notice that
     there's no user-email section at all. For logged in users, at least
     there are two entries; a user-email, and "Report anonymously".
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2826563002
Cr-Commit-Position: refs/heads/master@{#465057}
Committed: https://chromium.googlesource.com/chromium/src/+/8a299993ec8d4a6a5c58dfa30dacf58a5bc8aa0d

[afakhry, 2017-04-17 21:57:13]

Description was changed from

==========
Feedback: User email section should only be visible when useful

When user is not logged on, default is to report anonymously. No
user email is available for us. In this case, there's no need to
show the user email section. So we'll keep it hidden until a valid
email entry is added.

BUG=711357
TEST=login in guest mode, open feedback app (Alt+Shift+i), notice that
     there's no user-email section at all. For logged in users, at least
     there are two entries; a user-email, and "Report anonymously".
==========

to

==========
Feedback: User email section should only be visible when useful

When user is not logged on, default is to report anonymously. No
user email is available for us. In this case, there's no need to
show the user email section. So we'll keep it hidden until a valid
email entry is added.

BUG=711357
TEST=login in guest mode, open feedback app (Alt+Shift+i), notice that
     there's no user-email section at all. For logged in users, at least
     there are two entries; a user-email, and "Report anonymously".
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[afakhry, 2017-04-17 21:57:45]

The CQ bit was checked by afakhry@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-17 21:58:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[afakhry, 2017-04-17 21:58:53]

afakhry@chromium.org changed reviewers:
+ xiyuan@chromium.org

[afakhry, 2017-04-17 21:58:54]

Xiyuan, please take a look. Thanks!

[xiyuan, 2017-04-17 22:18:48]

lgtm

[afakhry, 2017-04-17 22:30:08]

The CQ bit was unchecked by afakhry@chromium.org

[afakhry, 2017-04-17 22:30:12]

The CQ bit was checked by afakhry@chromium.org

[commit-bot: I haz the power, 2017-04-17 22:30:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-17 23:20:35]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492468212625400, "parent_rev":
"ea81c77b5c2512e29f5b7fda56134778cac21a61", "commit_rev":
"8a299993ec8d4a6a5c58dfa30dacf58a5bc8aa0d"}

[commit-bot: I haz the power, 2017-04-17 23:20:52]

Description was changed from

==========
Feedback: User email section should only be visible when useful

When user is not logged on, default is to report anonymously. No
user email is available for us. In this case, there's no need to
show the user email section. So we'll keep it hidden until a valid
email entry is added.

BUG=711357
TEST=login in guest mode, open feedback app (Alt+Shift+i), notice that
     there's no user-email section at all. For logged in users, at least
     there are two entries; a user-email, and "Report anonymously".
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Feedback: User email section should only be visible when useful

When user is not logged on, default is to report anonymously. No
user email is available for us. In this case, there's no need to
show the user email section. So we'll keep it hidden until a valid
email entry is added.

BUG=711357
TEST=login in guest mode, open feedback app (Alt+Shift+i), notice that
     there's no user-email section at all. For logged in users, at least
     there are two entries; a user-email, and "Report anonymously".
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2826563002
Cr-Commit-Position: refs/heads/master@{#465057}
Committed:
https://chromium.googlesource.com/chromium/src/+/8a299993ec8d4a6a5c58dfa30dac...
==========

[commit-bot: I haz the power, 2017-04-17 23:20:53]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8a299993ec8d4a6a5c58dfa30dac...