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

Issue 765837 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Font awesome icon not showing

Reported by gmhaw...@gmail.com, Sep 15 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.91 Safari/537.36

Steps to reproduce the problem:
1. In private app, images describing issues attached
2. 
3. 

What is the expected behavior?
Awesome font icon to display on page load

What went wrong?
They used to display in Chrom 60, when I updated to Chrome 61 it stopped rendering the icon.  If I got to the dev console, and toggle the 'display' property for the element, the icon displays.  See attached files.

Did this work before? Yes 60

Chrome version: 61.0.3163.91  Channel: stable
OS Version: OS X 10.12.6
Flash Version: 

Please let me know when this is supposed to be fixed.  My email is gmhawash@gmail.com
 
chrome-bug.png
140 KB View Download
chrome-bug2.png
502 KB View Download

Comment 1 by gov...@chromium.org, Sep 15 2017

Cc: pbomm...@chromium.org ranjitkan@chromium.org
Labels: Needs-Triage-M61 M-61
Components: -UI Blink>CSS
Cc: krajshree@chromium.org
Labels: Needs-Feedback
gmhawash@ - Thanks for filing the issue...!!

Could you please provide a sample URL to test the issue from TE-end. This will help us in triaging the issue further.

Thanks...!!

Comment 4 by gmhaw...@gmail.com, Sep 19 2017

The app is locked down with password and browser certs.  I would be happy to set up an account for you to access, but I need a specific email address to send the certs to.  Should I use krajshree@chromium.org?  If not, please provide me with an email address.
Project Member

Comment 5 by sheriffbot@chromium.org, Sep 19 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "krajshree@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
Labels: Needs-Feedback
Hi gmhawash@,

Accepting credentials for bug repros is difficult for us for a variety of reasons.

instead of providing credentials, would it be possible to either:
* save the page on which the issue is occurring and attach the HTML to this issue; or
* save the page and strip out any sensitive information before attaching the HTML?

Comment 7 by gmhaw...@gmail.com, Sep 20 2017

Ok, I've saved the page from Chrome and attached a zip file below.
Chrome 61 issue.zip
583 KB Download
Project Member

Comment 8 by sheriffbot@chromium.org, Sep 20 2017

Cc: rjwright@chromium.org
Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "rjwright@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
Labels: Needs-Feedback
Can the page be simplified?

Is the icon shown in other browsers?

Comment 10 by gmhaw...@gmail.com, Sep 22 2017

Our app only works with Chrome, no other browser.  But, if you try Chrome 60 (and before) worked fine, but Chrome 61 does not.

Here is a video that demonstrates the issue https://youtu.be/u7fwGqlGByM.  There are two icons, one shows up immediately, and the other one does not.  As the video shows, I can make it show up by toggling the 'display' attribute.  On Chrome 60, it just shows up on page load.
Project Member

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

Cc: ericwilligers@chromium.org
Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "ericwilligers@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
Owner: ericwilligers@chromium.org
Assigning to Eric to confirm

Comment 13 by shend@chromium.org, Sep 25 2017

Status: Untriaged (was: Unconfirmed)
Confirmed 61.0.3163.100 (Official Build) (64-bit). Second icon doesn't show up until I toggle the display property.

Cannot repro on 63.0.3218.0 (Official Build) dev (64-bit). Second icon shows up immediately. Requesting triage to find the revision that fixed this issue.

Comment 14 by shend@chromium.org, Sep 25 2017

Labels: Needs-Bisect
Oops, I meant requesting a bisect.
Cc: e...@chromium.org robhogan@chromium.org
Components: Blink>HTML
Labels: -Pri-2 -Needs-Bisect -M-61 hasbisect-per-revision ReleaseBlock-Stable M-63 OS-Linux OS-Windows Pri-1
Status: Assigned (was: Untriaged)
Able to reproduce the issue on Windows 10, mac 10.12.6 and Ubuntu 14.04 using chrome stable version #61.0.3163.100 but the same is not reproducible in the latest canary #63.0.3223.8.

Reverse Bisect Information:
=====================
Good build: 63.0.3211.0  Revision(500753)
Bad Build : 63.0.3210.0  Revision(500529)

Change Log URL: 
https://chromium.googlesource.com/chromium/src/+log/14527543958d60b00d130bb0a06945d72b44966f..27c507212fd72de9f944ff001665c242a7785cce

From the above change log possible CL that fixed this issue:
Change-Id: I1ad11423182282066ede39a4e8da2bab0fbab1da
Reviewed-on: https://chromium-review.googlesource.com/655309

ericwilligers/eae@ - Could you please check and merge the fix to M63 if it is a valid canadidate.
Note: ccing reviewer(eae@) of the issue as the author(robhogan@) seems to be OOO from more than 30 days.
Also adding label ReleaseBlock-Stable as it seems to be a recent regression. Please feel free to remove the same if not appropriate.

Thanks...!!
Components: -Blink>CSS Blink>Layout
Owner: e...@chromium.org
Reassigning. This isn't a CSS bug.

Comment 17 by e...@chromium.org, Oct 3 2017

r500539 indeed seems like a likely change to have fixed this issue.
We're likely too late in the process to DI the fix into 62 by now.

gmhawash: A potential workaround until 63 is released would be to remove the "display: inline-block" style. That prevents it from disappearing without changing the layout in any way that I can detect.

krajshree: This is marked as RB-S for 63. Given that the issue does not occur in 63 that doesn't make much sense. Did you mean to mark it for M62?

Comment 18 by e...@chromium.org, Oct 4 2017

Owner: krajshree@chromium.org
To krajshree for clarification.
Labels: -M-63 M-62
Apologies for the milestone being marked to M-63. The issue is reproducible on latest beta #62.0.3202.45. Hence, changing the milestone to M-62.

Thanks...!!
Owner: e...@chromium.org
eae@, Just wondering, are we still targeting the fix to M62? If not please adjust the priority and other milestone labels.

Thank you!

Comment 22 by e...@chromium.org, Oct 10 2017

Labels: -ReleaseBlock-Stable -M-62 -Needs-Triage-M61 Needs-Triage-M62
Status: Fixed (was: Assigned)
Given that this is already broken in 61 and that a workaround is available I'm reluctant to integrate the fix in 62 given the risks involved.

Note that this *has* been fixed in 63.

Updating status accordingly.

Sign in to add a comment