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

Issue 7774012: renderer/extensions: Forward declare a couple of v8 classes in EventBindings header. (Closed)

Created:
9 years, 3 months ago by tfarina
Modified:
9 years, 3 months ago
CC:
chromium-reviews, Aaron Boodman, Erik does not do reviews, darin-cc_chromium.org, mihaip+watch_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

renderer/extensions: Forward declare a couple of v8 classes in EventBindings header. BUG=None TEST=None R=mpcomplete@chromium.org,darin@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98907

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -2 lines) Patch
M chrome/renderer/extensions/event_bindings.h View 2 chunks +6 lines, -2 lines 0 comments Download
M chrome/renderer/extensions/event_bindings.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
tfarina
9 years, 3 months ago (2011-08-29 13:43:28 UTC) #1
tfarina
+darin for review!
9 years, 3 months ago (2011-08-30 21:09:05 UTC) #2
darin (slow to review)
9 years, 3 months ago (2011-08-30 22:07:26 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698