diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-01 16:58:20 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-01 16:58:20 +0000 |
| commit | 8e0dca871966c15a08c23bfd890aad73ee3b8aa5 (patch) | |
| tree | 85d7f74e4b9fdff8becedb222c48500e3bc9cd4c /chrome/browser/autocomplete/autocomplete_edit.cc | |
| parent | b9e562e475b17f6c07a46de7afabdfa3164eafbf (diff) | |
| download | chromium_src-8e0dca871966c15a08c23bfd890aad73ee3b8aa5.zip chromium_src-8e0dca871966c15a08c23bfd890aad73ee3b8aa5.tar.gz chromium_src-8e0dca871966c15a08c23bfd890aad73ee3b8aa5.tar.bz2 | |
Make native button wrapper construction return the new wrapper rather than
setting it on the class. I did this for the textbox and I think it's a bit
better. The old way set it, but that function didn't add it to the view
hierarchy. This means that the pointer is technically "unowned" when the
CreateWrapper function returns.
This new way makes pointer ownership more clear. It becomes the caller's
problem, and the caller immediately inserts it into the view hierarchy which
then manages the lifetime.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/180041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@25043 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autocomplete/autocomplete_edit.cc')
0 files changed, 0 insertions, 0 deletions