Closed Bug 1487284 Opened 2 years ago Closed 2 years ago
Tools - Responsive Design Mode - on Refresh in RDM No CSS Rules Show
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:63.0) Gecko/20100101 Firefox/63.0 Build ID: 20180824192747 Steps to reproduce: Go to any website, inspect any element, switch into Responsive Design mode and then refresh the page. Actual results: Upon refresh of the page in RDM, there are no longer any css rules shown (elements), not even a message to show inspect any elements in some cases. Expected results: The CSS rules (elements) should be displaying as per normal after refresh.
I can reproduce this on Mac OS X 10.12, Windows 10 with the latest Nightly 63.0a1(2018-09-02). I did some tests and I found out that this is a regression, here is the regression range: Last good revision: 8b61609228c997c521223f9b6ea5ded55d31676f 69:08.57 INFO: First bad revision: 20d26085371c10a2b4a9a49a45b2f508bf2b2403 69:08.57 INFO: Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=8b61609228c997c521223f9b6ea5ded55d31676f&tochange=20d26085371c10a2b4a9a49a45b2f508bf2b2403 Dan, from the pushlog, seems that bug 1450923 caused this regression, can you please take a look?
Severity: normal → major
Status: UNCONFIRMED → NEW
Component: Untriaged → Responsive Design Mode
Ever confirmed: true
OS: Unspecified → All
Product: Firefox → DevTools
Hardware: Unspecified → All
I think the regression range must be wrong. All bug 1450923 did was change some initial favicons in the file that seeds bookmarks in a new profile. The STR here won't even be going anywhere near those and wouldn't be able to cause this sort of regression.
I did another regression range and here is the result: Last good revision: 484dc9b59dcaaedc039e5a851cdd6a997f713429 67:43.37 INFO: First bad revision: 3c1c7f965f0faadc2372c6585866cc36ce31413e 67:43.37 INFO: Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=484dc9b59dcaaedc039e5a851cdd6a997f713429&tochange=3c1c7f965f0faadc2372c6585866cc36ce31413e From the regression range seems that bug 1478995 might be the problem, Dan can you please take a look?
That bug seems unrelated to the STRs? Ovidiu, since you got different results with the two bisect, maybe the STRs are intermittent? Note that we also have similar Bug 1486259 on file.
See Also: → 1486259
Hi Julian, the issue is always reproducible with the steps, I can't figure out why the regression results are different.
Did a new bisect, pointing to Bug 1444132 (rev ace2f8e7a9c9), which can explain the issue.
Since Bug 1444132 is a patch queue, I verified that the issue did not happen before the first changeset, and started occuring after the last changeset (to make sure this was not a false positive). Since this is a massive refactor, this merely tells us that this issue probably a race condition related to network monitor changes. But we can't simply backout everything or find the issue based on this regression range I think.
The styles panel needs to fetch stylesheets, which involves asking the webconsole actor to the network-monitor actor to provide the content if it already has been downloaded. This relies on using the message manager to communicate between actors in parent and content processes. Switching to RDM makes the content message manager obsolete and I think the network-monitor actor is not notified of this change and misses the request from the console.
I have a fix, working on a test now
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Although this fixes the issue at hand, I am not sure what is the long term plan for spawnActorInParent vs setupInParent? Both methods seem to have similar roles and we should probably aim to have only one?
Comment on attachment 9006547 [details] Bug 1487284 - Update message manager in actors spawned in parent after browserSwap;r=yulia Yulia Startsev [:yulia] has approved the revision.
Attachment #9006547 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/7d0e529c9803 Update message manager in actors spawned in parent after browserSwap;r=yulia
Attachment #9006547 - Flags: approval-mozilla-beta?
Comment on attachment 9006547 [details] Bug 1487284 - Update message manager in actors spawned in parent after browserSwap;r=yulia Fix for a regresion, uplift approved for 63 beta 5
Attachment #9006547 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
I verified this issue on Mac Os X 10.12, Ubuntu 16.04 and Windows 10 x64 with FF Nightly 64.0a1(2018-09-11) and FF beta 63.0b5 and I can confirm the fix.
You need to log in before you can comment on or make changes to this bug.