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

Issue 4637002: Adds the ability for classes other than native control to process (Closed)

Created:
10 years, 1 month ago by sky
Modified:
9 years, 7 months ago
CC:
chromium-reviews, ben+cc_chromium.org
Visibility:
Public.

Description

Adds the ability for classes other than native control to process messages bounced to the parent and wires it up for autocompleteedit, which I'm going to need shortly. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=65612

Patch Set 1 #

Patch Set 2 : Tweaks #

Unified diffs Side-by-side diffs Delta from patch set Stats (+162 lines, -40 lines) Patch
M chrome/browser/autocomplete/autocomplete_edit_view_win.h View 7 chunks +19 lines, -1 line 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit_view_win.cc View 5 chunks +25 lines, -0 lines 0 comments Download
M views/controls/native_control_win.h View 4 chunks +8 lines, -10 lines 0 comments Download
M views/controls/native_control_win.cc View 1 4 chunks +8 lines, -7 lines 0 comments Download
M views/views.gyp View 2 chunks +4 lines, -0 lines 0 comments Download
A views/widget/child_window_message_processor.h View 1 chunk +50 lines, -0 lines 0 comments Download
A views/widget/child_window_message_processor.cc View 1 1 chunk +27 lines, -0 lines 0 comments Download
M views/widget/widget_win.cc View 5 chunks +21 lines, -22 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
sky
My first attempt at landing this broke a couple of ui tests. I'm trying again. ...
10 years, 1 month ago (2010-11-10 01:08:01 UTC) #1
Ben Goodger (Google)
10 years, 1 month ago (2010-11-10 19:11:23 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698