Bug 12293 - [S60] Saved Pages: CSS reference file not saved with page
Summary: [S60] Saved Pages: CSS reference file not saved with page
Status: CLOSED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 420+
Hardware: S60 Emulator S60 3rd edition
: P2 Normal
Assignee: Nobody
URL:
Keywords: PlatformOnly
Depends on:
Blocks:
 
Reported: 2007-01-16 13:02 PST by Hong Zhao
Modified: 2011-03-21 11:52 PDT (History)
1 user (show)

See Also:


Attachments
Save the import .css to when we generate Page data. (4.21 KB, patch)
2007-01-17 06:59 PST, Hong Zhao
zalan: review-
Details | Formatted Diff | Diff
Save the import .css when we get the data from page. (3.62 KB, patch)
2007-01-17 07:18 PST, Hong Zhao
zalan: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hong Zhao 2007-01-16 13:02:27 PST
TSW ID:  RFON-6SALS4  

1. Load http://TestSuite.nokia-boston.com/content/S60NG_SavePage/css.html
2. Save the page.
3. Close and re-open the browser
4. Open the saved page

Expected result: 
The save page function works properly with external css files.  Multiple external css files are saved; all css files should be included.

Actual result:
The page is open without the CSS properties
Comment 1 Hong Zhao 2007-01-17 06:59:09 PST
Created attachment 12505 [details]
Save the import .css to when we generate Page data.
Comment 2 Hong Zhao 2007-01-17 07:18:13 PST
Created attachment 12506 [details]
Save the import .css when we get the data from page.
Comment 3 zalan 2007-01-18 18:30:04 PST
Comment on attachment 12505 [details]
Save the import .css to when we generate Page data.

1,this fix fails to address the case where the imported stylesheet has an @import in it. 
2,if the css is 404, then  this function tries to load it again (not sure if that's the correct behaviour) but since the load is async, it will not be able to save the result of the load.
2, please fix the input path in the .mmp instead of doing this #include "../khtml/css/css_ruleimpl.h"
Comment 4 zalan 2007-01-18 18:30:47 PST
Comment on attachment 12506 [details]
Save the import .css when we get the data from page.

see above
Comment 5 Bradley Morrison 2008-04-09 11:31:58 PDT
Bulk update of s60 bugs - tagging and closing.
Comment 6 Bradley Morrison 2008-04-09 11:39:49 PDT
Bulk closing of all s60 platform bugs. 

Sorry for the noise!
Comment 7 Joel Parks 2011-03-21 11:52:59 PDT
re-purposing InTSW keyword for use by QtWebkit team