Bug 18362 – protocol 'data' does not change/clear the page title

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

[Ariya Hidayat]

Steps to reproduce:
(1) Open any website, e.g. slashdot.org
(2) The title is changed
(3) Put the following in the URL 
  data:text/html,<form></form>
and after that press Enter

What happens:
The title does not change

Expected:
The title should be emptied.

[Ariya Hidayat]

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.

[Ariya Hidayat]

> 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.

[Ariya Hidayat]

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.

[Ariya Hidayat]

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.

[Maciej Stachowiak]

Comment on attachment 20405 [details]
patch to clear the title when loading starts

Removing review flag since the patch is apparently withdrawn.