Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Issue 9389003: Merge 107242 - Web Inspector: fix setOuterHTML for the case that adds / removes duplicate. (Closed)

Created:
8 years, 10 months ago by pfeldman
Modified:
8 years, 10 months ago
Reviewers:
pfeldman
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1025/
Visibility:
Public.

Description

Merge 107242 - Web Inspector: fix setOuterHTML for the case that adds / removes duplicate. https://bugs.webkit.org/show_bug.cgi?id=78235 Reviewed by Yury Semikhatsky. Source/WebCore: * inspector/DOMEditor.cpp: (WebCore::DOMEditor::diff): (WebCore::DOMEditor::innerPatchChildren): (WebCore): (WebCore::nodeName): (WebCore::DOMEditor::dumpMap): * inspector/DOMEditor.h: (DOMEditor): LayoutTests: * inspector/elements/set-outer-html-2-expected.txt: * inspector/elements/set-outer-html-2.html: * inspector/elements/set-outer-html-expected.txt: * inspector/elements/set-outer-html.html: TBR=pfeldman@chromium.org BUG=113388 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=107557

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+96 lines, -134 lines) Patch
M LayoutTests/inspector/elements/set-outer-html.html View 1 chunk +0 lines, -9 lines 0 comments Download
M LayoutTests/inspector/elements/set-outer-html-2.html View 2 chunks +6 lines, -4 lines 0 comments Download
M LayoutTests/inspector/elements/set-outer-html-2-expected.txt View 8 chunks +43 lines, -26 lines 0 comments Download
M LayoutTests/inspector/elements/set-outer-html-expected.txt View 10 chunks +0 lines, -90 lines 0 comments Download
M Source/WebCore/inspector/DOMEditor.h View 1 chunk +3 lines, -0 lines 0 comments Download
M Source/WebCore/inspector/DOMEditor.cpp View 5 chunks +44 lines, -5 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
pfeldman
8 years, 10 months ago (2012-02-13 12:06:21 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698