Issue metadata
Sign in to add a comment
|
Extension content_script CSS is is not re-applied after DOM changes
Reported by
tom...@gmail.com,
Nov 21 2017
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:58.0) Gecko/20100101 Firefox/58.0 Steps to reproduce the problem: 1. Download and install the attached extension 2. Navigate to http://example.com 3. Wait 10 seconds and observe text color of added DIVs What is the expected behavior? All text on the page should have the same (red) text color. What went wrong? Text of DIVs added at different times is in different colors, even if only one and the same CSS rule applies to all of them (at one time). Did this work before? N/A Does this work in other browsers? N/A Chrome version: 64.0.3269.3 (Official Build) dev (64-bit) (cohort: Dev) Channel: dev OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: 27.0.0.187 This seems to only happen on Windows 7 (Intel i5, integrated graphics). Couldn't reproduce on Windows 10, Linux or OSX.
,
Nov 21 2017
Bisect info: 509091 (good) - 509105 (bad) https://chromium.googlesource.com/chromium/src/+log/17321e34..2fdeded5?pretty=fuller Suspecting r509096 = 30168b99604c48a32828f26b5acb2674beea1a5d = https://crrev.com/c/641294 by m.jethani@eyeo.com "Implement user style sheets" Landed in 64.0.3242.0
,
Nov 22 2017
Since this is specific to Windows 7, it would be unlikely that this is because of https://crrev.com/c/641294. Nevertheless I'll take a look. @woxxom it looks like you accidentally removed a bunch of people from the cc list of https://crrev.com/c/641294
,
Nov 22 2017
,
Nov 22 2017
As per comment#3 suspecting Reviewed-on: https://chromium-review.googlesource.com/641294. @m.jethani: Please confirm the above issue and feel free to re-assign if it is not related to your change. Thanks!
,
Nov 22 2017
> Since this is specific to Windows 7, it would be unlikely that this is because Sorry for the confusion, turns out this probably isn't Windows 7 specific, but Dev 64.0.3269.3 specific (which was only tested on Windows 7). Just verified everything works as expected in Beta 63.0.3239.59 on Windows 7, and can confirm Dev on other platforms tomorrow.
,
Nov 22 2017
>@woxxom it looks like you accidentally removed a bunch of people from the cc list of https://crrev.com/c/641294 Sorry about that. Thought I removed them from my message only. If the CL's list got modified I'd say that tracker has a serious priviledge management problem as I'm not affiliated neither to that CL nor to any project at all.
,
Nov 29 2017
tomica@ can you check with the latest Dev? We just switched back to author style sheets as the default, this should fix the regression. It would still be an issue with user style sheets, but perhaps we can file that one separately.
,
Dec 28 2017
I have just verified and this is no longer an issue. nainar@ is it appropriate to mark this is as a duplicate of issue 632009 ?
,
Jan 1 2018
Marking as Fixed should do it. Duplicate indicates it is the same issue as reported elsewhere.
,
Jan 2 2018
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by tom...@gmail.com
, Nov 21 2017765 bytes
765 bytes Download