818431 - [style editor] Since the Toolbox, sometimes, the source editor in the style editor doesn't show up

Status: RESOLVED WORKSFORME

(DevTools :: Style Editor, defect, P2)

Description

[Paul Rouget [:paul]]

No STR. To investigate.

Comment 1

[Girish Sharma [:Optimizer]]

So even you saw this once ?

Comment 2

[Paul Rouget [:paul]]

(In reply to Girish Sharma [:Optimizer] from comment #1)
> So even you saw this once ?

?

Comment 3

[Girish Sharma [:Optimizer]]

That the style editor did not load. 
What I saw once that on Facebook, after trying to load for a while, Style Editor said that no styles exist on the page, want to make one ?

Comment 4

[Mihai Sucan [:msucan]]

This is what could be breaking bug 808264.

Comment 5

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

I managed to replicate Optimizer's Facebook issue once. It loaded okay, including the stylesheet and then the editor went blank.

This reminded me of another old issue with the style editor. Sometimes it gets loaded twice in quick succession and the second load sometimes breaks the editor (which makes sense if it was previously half initialized).

We could never find reliable set of STR but I am fairly certain that fixing the double load will fix the error.

I will take a look.

Comment 6

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

I finally have reliable STR:
1. Open about:home (or any other page).
2. Open the style editor: It will be loaded just fine.
3. Refresh the page (or navigate to a new page).

- If the page was refreshed then the inline stylesheets are now missing and the editor is empty.
- If a new page was loaded then the editor is blank.

Comment 7

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

I will be away until the middle of January 2013 so I am unassigning myself from this bug.

Comment 8

[Heather Arthur [:harth]]

I fixed some related bugs recently (bug 826180 and bug 826191). Can you guys still reproduce your problems?

Comment 9

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Works perfectly fine now, great job.