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

Issue 237513004: Implemented custom support for scrollLeft/scrollTop in Element (Closed)

Created:
6 years, 8 months ago by Leaf
Modified:
6 years, 8 months ago
Reviewers:
vsm
CC:
reviews+dom_dartlang.org
Visibility:
Public.

Description

Implemented custom support for scrollLeft/scrollTop in Element Changes to the IDL spec made scrollLeft/scrollTop custom. This patch implements the old behavior (passing in an integer), but not the new extensions (passing in a dictionary). BUG=https://code.google.com/p/dart/issues/detail?id=18062 R=vsm@google.com Committed: https://src.chromium.org/viewvc/multivm/branches/1650/blink?view=rev&revision=171516

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -16 lines) Patch
M Source/bindings/dart/custom/DartElementCustom.cpp View 2 chunks +28 lines, -16 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Leaf
6 years, 8 months ago (2014-04-14 23:54:40 UTC) #1
vsm
lgtm
6 years, 8 months ago (2014-04-14 23:55:42 UTC) #2
Leaf
6 years, 8 months ago (2014-04-14 23:58:15 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r171516 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698