LGTM++

Although I don't fully understand this changelist (given that I haven't seen the
corresponding patch to the Chromium source that works with this Gears
changelist) and would normally still have reservations (e.g. the "moot point"
regarding GearsHtmlEventListener), I'm approving this CL, on the understanding
that (1) not having Gears code to point to may be blocking you on working
through the Chromium review process, and (2) this "v1 API" implementation will
probably be left on the cutting room floor in favor of the "v2 API", and it's
not worth the time to polish code we'll just delete anyway.

-- 
To respond, reply to this email or visit 
http://mondrian.corp.google.com/10012347

Reply via email to