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

Issue 11118025: Extract common logic from Backend.BaseClassName(). (Closed)

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

Description

Extract common logic from Backend.BaseClassName(). R=antonm@google.com Committed: https://code.google.com/p/dart/source/detail?r=13742

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -37 lines) Patch
M lib/html/scripts/systemhtml.py View 2 chunks +13 lines, -16 lines 0 comments Download
M lib/html/scripts/systemnative.py View 2 chunks +3 lines, -21 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
podivilov
8 years, 2 months ago (2012-10-12 17:25:23 UTC) #1
Anton Muhin
8 years, 2 months ago (2012-10-12 17:43:33 UTC) #2
lgtm, but the whole logic might be worth a separate method

Powered by Google App Engine
This is Rietveld 408576698