Bug 32671 – REGRESSION(4.0.4-TOT):Web Inspector: Incorrect Doctype

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Jay Robinson]

Created attachment 45084 [details]
Screenshot of Apple.com/support inspected by Web Inspector with actual source code window behind the browser window.

All HTML source code viewed through the Web Inspector will show an HTML5 doctype instead of the doctype actually specified. 

Tested with various doctypes with the same results. This has been a problem for at least the last several builds.

For example, Apple.com/support specifies HTML 4.01 transitional. However, the Web Inspector shows an HTML doctype on the first line of the HTML source code. 

Screenshot attached.

[Timothy Hatcher]

I wonder if we are showing the inspector's doctype and not the page's…

[Timothy Hatcher]

I see the correct doctype in Safari 4.0.4.

[Pavel Feldman]

Created attachment 45094 [details]
[PATCH] The fix.

I did not copy those via dom agent...

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45094 [details] without any errors.

[Pavel Feldman]

Created attachment 45095 [details]
[PATCH] Same with tests.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45095 [details] without any errors.

[WebKit Commit Bot]

Comment on attachment 45095 [details]
[PATCH] Same with tests.

Rejecting patch 45095 from commit-queue.

Unexpected failure when landing patch!  Please file a bug against bugzilla-tool.
Failed to run "['WebKitTools/Scripts/bugzilla-tool', '--status-host=webkit-commit-queue.appspot.com', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', '--quiet', '45095']" exit_code: 1
Last 500 characters of output:
ut))
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/modules/statusbot.py", line 73, in update_status
    response = self.browser.submit()
  File "build/bdist.macosx-10.5-i386/egg/mechanize/_mechanize.py", line 547, in submit
  File "build/bdist.macosx-10.5-i386/egg/mechanize/_mechanize.py", line 209, in open
  File "build/bdist.macosx-10.5-i386/egg/mechanize/_mechanize.py", line 261, in _mech_open
mechanize._response.httperror_seek_wrapper: HTTP Error 500: Internal Server Error

[Pavel Feldman]

Comment on attachment 45095 [details]
[PATCH] Same with tests.

Another attempt

[Eric Seidel (no email)]

I think Adam is working on a fix for this class of bug.  AppEngine (which is where the queue status server http://webkit-commit-queue.appspot.com/ is hosted) commonly gives us back 500 errors.  We should silently ignore them and try again.

[WebKit Commit Bot]

Comment on attachment 45095 [details]
[PATCH] Same with tests.

Clearing flags on attachment: 45095

Committed r52279: <http://trac.webkit.org/changeset/52279>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Adam Barth]

> I think Adam is working on a fix for this class of bug.

This is a different kind of bug.  Basically, by piping the status-host to bugzilla-tool, you made it possible for it to throw a new kind of error.  We need to update the error handling to take that into account.

[Alexey Proskuryakov]

[phone]  Pavel Feldman  <[e-mail]>

Is that your preferred e-mail address?

[Pavel Feldman]

(In reply to comment #13)
> [phone]  Pavel Feldman  <[e-mail]>
> 
> Is that your preferred e-mail address?

Heh. Thanks for heads up. This is a second time my config goes crazy.