Issue metadata
Sign in to add a comment
|
DevTools cannot "Add new style rule" to css files for local webpages
Reported by
billchen...@gmail.com,
Mar 4 2018
|
||||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.106 Safari/537.36 Steps to reproduce the problem: 1. Create any HTML webpage and run it on Chrome, the URL will be like "file:///C:/test.htm". 2. Open DevTools -> select any HTML element -> Click the icon of "Add new style rule", if choosing "inspector-stylesheet", everything is fine and styles can be added successfully. HOWEVER, if choosing any specific css file like the screenshot below shows, then it won't work, the style will disappear. 3. If you open the same webpage with URL like "http://localhost/test.htm", everything works fine. 4. The bug exists since Chrome 64. For chrome 64, "Add new style rule" doesn't work for local page, then I reported the bug (issue id: 814937) to the Chrome team, they fix it on Chrome 65 beta. But when I test it, I still encounter the issue here. What is the expected behavior? What went wrong? The style cannot be added to any other css file except the default "Inspector-Stylesheet" Did this work before? Yes Chrome 63 Chrome version: 65.0.3325.106 Channel: beta OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: I reported the bug below, the Chrome team said they have fixed it on Chrome 65 beta. When I test it and found not completely fix it. Here is the original bug's link: https://bugs.chromium.org/p/chromium/issues/detail?id=814937#c20
,
Mar 5 2018
Able to reproduce this issue on latest stable 64.0.3282.186, on latest beta 65.0.3325.106 and latest canary 67.0.3361.0 using Mac 10.13.3, Ubuntu 14.04 and Windows 10. i.e; Unable to add new style rule when css is selected. NOTE: Issue 814937 mentioned in comment#0 is fixed on above builds. Good Build: 64.0.3280.0 Bad Build: 64.0.3281.0 You are probably looking for a change made after 520004 (known good), but no later than 520005 (first known bad). CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/447db371759f0d9638769a7b22b9560a1b2d5da5..a4ebe08c91e29140e700c7bae9b94f27a786d1ca Reviewed-on: https://chromium-review.googlesource.com/783911 Suspecting same from changelog. As @ meade is no longer working on chrome assigning to @lushnikov from 814937. @lushnikov: Please confirm the bug and help in re-assigning if it is not related to your change. Adding RB-Stable as this is a recent regression. Please change if not the case. Thanks!
,
Mar 5 2018
Per comment
,
Mar 5 2018
,
Mar 5 2018
Sorry, duplicated this one by mistake. We'll take a look. Thanks for filing this issue! Not an M65 blocker given M64 presence, but we'll fix for M66.
,
Mar 19 2018
Friendly ping to get an update on this issue as it is marked as stable blocker. Thanks..!
,
Mar 26 2018
Gentle ping to get an update on this issue. Thanks..!
,
Mar 26 2018
Can anybody know that when this bug will be fixed? which version of Chrome will contain the fixed feature?
,
Mar 30 2018
lushnikov@, do we have further update on this?
,
Apr 2 2018
Just a heads up, M66 Stable cut is on April 12th, 10 days away. This issue is marked as RB-Stable for 66. Please make sure to address this issue prior to stable cut. Thanks!
,
Apr 9 2018
Friendly ping to get an update on this issue as it is marked as stable blocker & M66 Stable cut is on April 12th. Thanks..!
,
Apr 9 2018
Reminder: Please note that M66 Stable is only 7 days away. This bug has been marked as ReleaseBlock Stable for M66. So please take a look and appropriately address this bug.
,
Apr 13 2018
,
Apr 13 2018
It is so disappointing that such an important bug needs taking 2+ months to fix it.
,
Apr 19 2018
Today Google Chrome 66 is just released, I update my web browser immediately, just hope this bug has been fixed. I crossed my finger and pray the annoying bug can disappear. However, the bug is still there. Please developers take actions faster to kill this bug, Please.
,
Apr 19 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/85535060d2e16341b2bde83e21e2a41a7ad35509 commit 85535060d2e16341b2bde83e21e2a41a7ad35509 Author: Andrey Lushnikov <lushnikov@chromium.org> Date: Thu Apr 19 21:24:31 2018 DevTools: fix CSS.addRule protocol method for file:// urls The previous attempt to fix the bug was in crrev.com/539166: the patch allowed adding rules to the inspector style sheets. However, it didn't take into account adding rules into the already-existing style sheets. This patch fixes CSS.addRule altogether, allowing CSSOM access if the operation is happenning from-inside InspectorMutationScope. Our tests didn't catch the regression since we run with --allow-file-access-from-files flag. BUG= 818518 R=dgozman Change-Id: Ic6b198a5ef7abbabcef671edbefb5a9b14fde531 Reviewed-on: https://chromium-review.googlesource.com/1018526 Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org> Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Reviewed-by: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#552161} [modify] https://crrev.com/85535060d2e16341b2bde83e21e2a41a7ad35509/third_party/blink/renderer/core/css/css_style_sheet.cc [modify] https://crrev.com/85535060d2e16341b2bde83e21e2a41a7ad35509/third_party/blink/renderer/core/css/css_style_sheet.h [modify] https://crrev.com/85535060d2e16341b2bde83e21e2a41a7ad35509/third_party/blink/renderer/core/inspector/InspectorCSSAgent.cpp
,
Apr 20 2018
@billchengreat: sorry for this taking so long. The fix will be available on tomorrow's Canary.
,
Apr 20 2018
Able to reproduce the issue on reported version 65.0.3325.106, hence verifying the fix on latest canary 68.0.3401.0 using Mac 10.13.3, Windows 10 and Ubuntu 14.04. Observing successful adding of style rule. Attaching screencast for reference. As fix is working as expected, adding Verified labels. Thanks!
,
Apr 25 2018
M67 Stable promotion is coming soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge into the release branch ASAP. Thank you.
,
Apr 30 2018
I don't think this is safe to merge into Stable: - the merge will have to be manual due to Great Blink Move - we failed to come up with a decent test coverage for this due to the limitations of our test infrastructure.
,
Apr 30 2018
[Auto-generated comment by a script] We noticed that this issue is targeted for M-67; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-67 label, otherwise remove Merge-TBD label. Thanks.
,
Apr 30 2018
,
Jun 1 2018
The bug was promised to be fixed in version 67. I have waited for so long and finally Chrome 67 is released today. I am so excited to check if the bug has been killed or not, however, it still ugly exists in the latest version 67. It appears Chrome development team needs 1 century to fix such a small bug, and the promise is broken again and again. Please wake up, Google Chrome team, don't sleep on the desk too long, you guys need to WORK! |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by krajshree@chromium.org
, Mar 5 2018