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

Issue 9125011: Fix for Issue 1075 (Closed)

Created:
8 years, 11 months ago by sra1
Modified:
8 years, 11 months ago
Reviewers:
vsm
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix for Issue 1075 Don't use ImplementationFunction - that is the C++ name. There used to be a problem with IDBCursor.delete - some browsers didn't like the call foo.delete() The way to fix this if it reappears is to generate foo['delete']() instead. BUG=1075 TEST= Committed: https://code.google.com/p/dart/source/detail?r=3060

Patch Set 1 #

Patch Set 2 : merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -22 lines) Patch
M client/dom/frog/dom_frog.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M client/dom/generated/src/frog/Console.dart View 1 chunk +1 line, -1 line 0 comments Download
M client/dom/generated/src/interface/Console.dart View 1 chunk +1 line, -1 line 0 comments Download
M client/dom/generated/src/wrapping/_ConsoleWrappingImplementation.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M client/dom/generated/wrapping_dom.js View 7 chunks +8 lines, -8 lines 0 comments Download
M client/dom/generated/wrapping_dom_externs.js View 6 chunks +5 lines, -6 lines 0 comments Download
M client/dom/idl/dart/dart.idl View 1 chunk +2 lines, -0 lines 0 comments Download
M client/dom/scripts/dartgenerator.py View 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
sra1
8 years, 11 months ago (2012-01-06 22:13:16 UTC) #1
vsm
8 years, 11 months ago (2012-01-06 22:15:40 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698