Bug 62272

Summary: Web Inspector: editing body multiplies head
Product: WebKit Reporter: Andrey Kosyakov <caseq>
Component: Web Inspector (Deprecated)Assignee: Pavel Feldman <pfeldman>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
[Patch] Same with ability to set document's outer html. yurys: review+

Description Andrey Kosyakov 2011-06-08 03:49:04 PDT
1. Navigate to data:text/html,<div>
2. Open inspector, switch to Elements panel
3. Select <body>, then choose "Edit as HTML" from the context menu
4. Press ctrl+enter to commit the changes
5. Observe two <head>s in the document
(repeating these actions further yields more heads)
Comment 1 Pavel Feldman 2012-01-15 02:40:11 PST
Created attachment 122563 [details]
Patch
Comment 2 Pavel Feldman 2012-01-15 02:43:03 PST
*** Bug 76145 has been marked as a duplicate of this bug. ***
Comment 3 Pavel Feldman 2012-01-15 10:11:09 PST
Created attachment 122568 [details]
[Patch] Same with ability to set document's outer html.
Comment 4 Yury Semikhatsky 2012-01-16 05:13:19 PST
Comment on attachment 122568 [details]
[Patch] Same with ability to set document's outer html.

View in context: https://bugs.webkit.org/attachment.cgi?id=122568&action=review

> Source/WebCore/inspector/DOMEditor.h:46
>  class HTMLElement;

Remove this as well.

> LayoutTests/inspector/elements/set-outer-html-body.html:52
> +            // User console.log for output since body has been overwritten.

Remove this comment?

> LayoutTests/inspector/elements/set-outer-html-body.html:68
> +Tests DOMAgent.setOuterHTML invoked on body tag.

Missing bug link.
Comment 5 Pavel Feldman 2012-01-16 05:35:24 PST
Committed r105058: <http://trac.webkit.org/changeset/105058>