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

Issue 8883032: Merge 102317 - https://bugs.webkit.org/show_bug.cgi?id=74038 (Closed)

Created:
9 years ago by Dmitry Lomov (no reviews)
Modified:
9 years ago
Reviewers:
Dmitry Lomov
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/963/
Visibility:
Public.

Description

Merge 102317 - https://bugs.webkit.org/show_bug.cgi?id=74038 [V8][Chromium] Support legacy argument order in window.postMessage/window.webkitPostMessage. Reviewed by David Levin. Source/WebCore: * bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::isLegacyTargetOriginDesignation): (WebCore::handlePostMessageCallback): LayoutTests: * fast/dom/Window/window-postmessage-args-expected.txt: * fast/dom/Window/window-postmessage-args.html: New tests for legacy argument order. * platform/chromium/fast/dom/Window/window-postmessage-args-expected.txt: TBR=caseq@chromium.org ISSUE=http://code.google.com/p/chromium/issues/detail?id=106797 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=102386

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -6 lines) Patch
M LayoutTests/ChangeLog View 1 chunk +11 lines, -0 lines 0 comments Download
M LayoutTests/fast/dom/Window/window-postmessage-args.html View 1 chunk +3 lines, -0 lines 0 comments Download
M LayoutTests/fast/dom/Window/window-postmessage-args-expected.txt View 2 chunks +6 lines, -0 lines 0 comments Download
M LayoutTests/platform/chromium/fast/dom/Window/window-postmessage-args-expected.txt View 2 chunks +6 lines, -0 lines 0 comments Download
M Source/WebCore/ChangeLog View 1 chunk +11 lines, -0 lines 0 comments Download
M Source/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp View 2 chunks +22 lines, -6 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Dmitry Lomov (no reviews)
9 years ago (2011-12-08 22:28:24 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698