Summary: | protocol 'data' does not change/clear the page title | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ariya Hidayat <ariya.hidayat> | ||||
Component: | Page Loading | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED WONTFIX | ||||||
Severity: | Normal | CC: | ariya.hidayat, mitz | ||||
Priority: | P3 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Other | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Ariya Hidayat
2008-04-08 06:10:50 PDT
Created attachment 20405 [details]
patch to clear the title when loading starts
This is a potential (unfortunately, rather ugly) workaround. It ensures that the loader's client gets an empty string so that if there no <title> element in the document, the title which is displayed by the loader's client is properly cleared at the beginning.
In Safari, the window title changes to the URL, so I am not sure why this patch is needed. > In Safari, the window title changes to the URL, so I am not sure why
> this patch is needed.
I think it is Safari-specific.
It is debatable whether the title should change to an empty or the URL. But the point is that WebKit does not notify the loader/client that the currently loaded URI does not have a title.
Another proposal for solution: IF (1) the document has no <title> nor JavaScript that sets the title (2) the loader never calls dispatchDidReceiveTitle() of its client (3) the document finished loading THEN before calling dispatchDidFinishLoading() of the loader's client, it should also call dispatchDidReceiveTitle() with an empty title. (In reply to comment #3) > WebKit does not notify the loader/client that the currently > loaded URI does not have a title. It seems sensible to notify when something has changed. I am not sure what is the change reflected by the proposed notification. On #webkit, Mitz Pettel suggested that this should be taken care by the platform loader client, i.e. it should assume that the currently loaded URI/document has no title. IMO this is likely the optimal approach. Comment on attachment 20405 [details]
patch to clear the title when loading starts
Removing review flag since the patch is apparently withdrawn.
|