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

Issue 767762 link

Starred by 0 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Missing space on Media Capture notifications

Reported by alculqui...@gmail.com, Sep 22 2017

Issue description

Steps to reproduce the problem:
1. Go to https://permission.site 
2. Click on "Camera"
3. Grant permission
4. See notification

What is the expected behavior?

What went wrong?
The text isn't properly formatted: an space is missing between the 2 phrases.

Did this work before? N/A 

Chrome version: 61.0.3163.98  Channel: n/a
OS Version: 7.1.1
Flash Version:
 
Cc: msrchandra@chromium.org rbasuvula@chromium.org nyerramilli@chromium.org ligim...@chromium.org sandeepkumars@chromium.org
Labels: Needs-triage-Mobile Triaged-Mobile Needs-Feedback
Thanks for filing the issue! Unable to reproduce the issue in Android. Able see the spaces between the 2 phrases.

Steps Followed:
1. Launched Browser
2. Navigated to https://permission.site.
3. Clicked on camera and click on allow 
Observed that notifications displaying with spaces.

Chrome versions tested:
61.0.3163.98
63.0.3214.0

OS:
Android 7.1.2

Android Devices:
7.1.2 Pixel XL Build/N2G48E

@alculquicondor: Could you please let me know if i have missed anything and if possible, please provide the device details & screen shot/cast of the issue which would help us to triage the issue further.

Thank You!
It might be a bit difficult to spot.
Screenshot_2017-09-22-00-37-25-01.jpeg
69.3 KB View Download
Project Member

Comment 3 by sheriffbot@chromium.org, Sep 22 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "rbasuvula@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: nyquist@chromium.org
CC'ed reviewer from the above CL for further input's on this issue.

Thanks!!
Labels: Needs-Feedback
@nyquis: Could you please take a look at this and give us an update?

Thanks!!
Status: Started (was: Unconfirmed)
OP wrote a fix for this.
Project Member

Comment 8 by bugdroid1@chromium.org, Oct 17 2017

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

commit 5e5f61c2243af61972bbf94769debf1bcf1c49a7
Author: Aldo Culquicondor <alculquicondor@gmail.com>
Date: Tue Oct 17 16:03:13 2017

Add missing space on Media Capture notification

Bug: 767762
Change-Id: Ia6cd239f949ceed7706a63a23b53f186eb94e556
Reviewed-on: https://chromium-review.googlesource.com/678514
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509398}
[modify] https://crrev.com/5e5f61c2243af61972bbf94769debf1bcf1c49a7/AUTHORS
[modify] https://crrev.com/5e5f61c2243af61972bbf94769debf1bcf1c49a7/chrome/android/java/src/org/chromium/chrome/browser/media/MediaCaptureNotificationService.java

Status: Available (was: Started)
This issue has been marked as started, but has no owner. Making available.

Sign in to add a comment