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

Issue 11791021: Merge 137964 (Closed)

Created:
7 years, 11 months ago by haraken
Modified:
7 years, 11 months ago
Reviewers:
commit-queue
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1364/
Visibility:
Public.

Description

Merge 137964 > Regression causing DOM objects to have unstable NPObject* references with v8 bindings > https://bugs.webkit.org/show_bug.cgi?id=104921 > > Source/WebCore: > > Patch by Matthew Dempsky <mdempsky@google.com>; on 2012-12-17 > Reviewed by Kentaro Hara. > > Fix regression introduced by changeset 135804 resulting in > unstable NPObject* references for v8 objects. In the iter != > v8NPObjectMap->end() code path, objectVector was left unassigned > if the for loop terminated without returning. > > Also, V8Object::GetIdentityHash() is documented as not being guaranteed > as unique. As such, don't ASSERT() that two objects with the same hash > must therefor be the same object. > > Tests: plugins/npruntime/embed-property-iframe-equality.html > > * bindings/v8/NPV8Object.cpp: > (WebCore::npCreateV8ScriptObject): Fix. > > LayoutTests: > > Patch by Mathew Dempsky <mdempsky@google.com>; on 2012-12-17 > Reviewed by Kentaro Hara. > > Add variant of embed-property-equality test to verify that the > test still passes when the object being tested for equality > has already been remembered by a plugin from a different > JavaScript context. > > * plugins/npruntime/embed-property-iframe-equality.html: Added. > * plugins/npruntime/embed-property-iframe-equality-expected.txt: Added. TBR=commit-queue@webkit.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=138940

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+533 lines, -1 line) Patch
M LayoutTests/ChangeLog View 1 chunk +201 lines, -0 lines 0 comments Download
A + LayoutTests/plugins/npruntime/embed-property-iframe-equality.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/plugins/npruntime/embed-property-iframe-equality-expected.txt View 0 chunks +-1 lines, --1 lines 0 comments Download
M Source/WebCore/ChangeLog View 1 chunk +332 lines, -0 lines 0 comments Download
M Source/WebCore/bindings/v8/NPV8Object.cpp View 1 chunk +2 lines, -3 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
haraken
7 years, 11 months ago (2013-01-07 15:10:00 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698