Bug 62272 - Web Inspector: editing body multiplies head
Summary: Web Inspector: editing body multiplies head
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Pavel Feldman
URL:
Keywords:
: 76145 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-06-08 03:49 PDT by Andrey Kosyakov
Modified: 2012-01-16 05:35 PST (History)
10 users (show)

See Also:


Attachments
Patch (15.67 KB, patch)
2012-01-15 02:40 PST, Pavel Feldman
no flags Details | Formatted Diff | Diff
[Patch] Same with ability to set document's outer html. (16.52 KB, patch)
2012-01-15 10:11 PST, Pavel Feldman
yurys: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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>