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

Issue 10821010: Automatically support fancy Console methods. (Closed)

Created:
8 years, 5 months ago by Anton Muhin
Modified:
8 years, 4 months ago
Reviewers:
podivilov
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Automatically support fancy Console methods. TBR=podivilov@chromium.org Committed: https://code.google.com/p/dart/source/detail?r=9887

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -12 lines) Patch
M lib/dom/scripts/systemnative.py View 3 chunks +7 lines, -12 lines 2 comments Download

Messages

Total messages: 3 (0 generated)
Anton Muhin
8 years, 5 months ago (2012-07-25 15:29:03 UTC) #1
podivilov
https://chromiumcodereview.appspot.com/10821010/diff/1/lib/dom/scripts/systemnative.py File lib/dom/scripts/systemnative.py (right): https://chromiumcodereview.appspot.com/10821010/diff/1/lib/dom/scripts/systemnative.py#newcode684 lib/dom/scripts/systemnative.py:684: arguments = arguments[:-1] It does the opposite to DatabaseBuilder.AddMissingArguments, ...
8 years, 4 months ago (2012-08-21 12:03:27 UTC) #2
Anton Muhin
8 years, 4 months ago (2012-08-21 12:04:33 UTC) #3
https://chromiumcodereview.appspot.com/10821010/diff/1/lib/dom/scripts/system...
File lib/dom/scripts/systemnative.py (right):

https://chromiumcodereview.appspot.com/10821010/diff/1/lib/dom/scripts/system...
lib/dom/scripts/systemnative.py:684: arguments = arguments[:-1]
I'd be glad to, but I need proper signatures for all the backends and interface
generators.

On 2012/08/21 12:03:27, podivilov wrote:
> It does the opposite to DatabaseBuilder.AddMissingArguments, right? Could you
> please remove both hacks instead?

Powered by Google App Engine
This is Rietveld 408576698