Bug 13661 – REGRESSION (r21367): Local css ignored until page is refreshed

• Status: RESOLVED
• Resolution: FIXED
• Component: Page Loading

[Kevin M. Dean]

It seems that local css file as set under the advanced Preferences is no longer being applied in r21368. I use Safari Enhancer to remove underlines from link defaults and it no longer is loading the stylesheet. The activity window doesn't list it. Reverting to Safari works and shows the css file in the activity window.

[Matt Lilek]

This works for me with r21371

[Matt Lilek]

Try removing Safari Enhancer and then try setting a local stylesheet and see if it's still broken.

[Kevin M. Dean]

(In reply to comment #1)
> This works for me with r21371
> 

I only get the nightly builds so it's possible it's already fixed

[Kevin M. Dean]

(In reply to comment #2)
> Try removing Safari Enhancer and then try setting a local stylesheet and see if
> it's still broken.
> 

There's nothing to remove with Safari Enhancer. It simply automates the connection to the local css file, doing the same as manually setting it via the preferences. I tried manually setting the pref anyway with no change.

[Kevin M. Dean]

Interestingly, refreshing a page applies the css, but  not when it's first viewed after launching webkit. Once refreshed on one site, it seems to work on other sites too.

[Matt Lilek]

(In reply to comment #5)
> Interestingly, refreshing a page applies the css, but  not when it's first
> viewed after launching webkit. Once refreshed on one site, it seems to work on
> other sites too.
> 

Yeah, I definitely see that with r21371

[Matt Lilek]

Looks like more fallout from Maciej's change in r21367 (nothing else in the r21359-r21368 range looks like it could have caused it).

[Maciej Stachowiak]

I cannot reproduce the described bug.

[Kevin M. Dean]

I'm re-opening because it hasn't been resolved as of r21473 and stills fails the same.

Just so the steps are clear, get the css file from the following attachment.
Then go to Preferences-->Advanced-->Style Sheet and select the attachment css file.
Then quit and re-launch Webkit. Make sure you're not running any other copies of the Webkit app. That seems to have an effect on the css load.
After the startup page, go immediately to <http://www.imdb.com/>
If it's failing the links should appear underlined. Refresh the browser and the underlines go away like expected due to the css file. Restart Webkit and the underlines are back until you refresh again.

[Kevin M. Dean]

Created attachment 14564 [details]
linkunder.css

[Kevin M. Dean]

Here's a twist. The problem only occurs if it's the latest Nightly build that loads the Webkit Nightly start page. I'm currently using a previous version because the latest Nightly crashes too much. Doing so doesn't load the start page and I don't see the link underline issue. Even if I go to previous builds that definitely had the problem... they no longer have the problem. If I run the latest again, problem is back. So, it's looking like the problem is triggered by the initial load of the start page in the current nightly build only. Weird.

[Kevin M. Dean]

OK, it appears this problem has evolved into the issue I'm seeing in Bug 13880. Now if I have a Stylesheet set in the Advanced prefs, Webkit crashes on launch. Setting the stylesheet to none removes the problem. Even setting it to a blank css file brings back the crashing.