Get rid of the EnableClose() infrastructure in Widget.

Get rid of the EnableClose() infrastructure in Widget.

Only one place was left using it (FirstRunSearchEngineView), and Brian said that didn't need to use it if it would just pick the first of the engine choices (prior to randomization) as the window was closed.

http://crbug.com/102581
TEST=unittests, and: run organic first run chrome, close the search engine selector, the Google should be the default. run organic first run Chrome, choose a non-Google engine, it should be the default.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111096

[Ben Goodger (Google), 2011-11-22 00:01:07]

mirandac: first_run_search_view.cc/h, _unittest.cc
sky: everything else (testing profile, views api changes)

[Miranda Callahan, 2011-11-22 00:11:58]

On 2011/11/22 00:01:07, Ben Goodger (Google) wrote:
> mirandac: first_run_search_view.cc/h, _unittest.cc
> sky: everything else (testing profile, views api changes)

first_run_search_engine_* LGTM

[sky, 2011-11-22 00:37:32]

LGTM

http://codereview.chromium.org/8618001/diff/10004/chrome/browser/ui/views/fir...
File chrome/browser/ui/views/first_run_search_engine_view_unittest.cc (right):

http://codereview.chromium.org/8618001/diff/10004/chrome/browser/ui/views/fir...
chrome/browser/ui/views/first_run_search_engine_view_unittest.cc:8: #include
"chrome/browser/ui/views/first_run_search_engine_view.h"
nit: include this first (just like for first_run_search_engine_view.cc).

http://codereview.chromium.org/8618001/diff/10004/chrome/browser/ui/views/fir...
chrome/browser/ui/views/first_run_search_engine_view_unittest.cc:25: // Set a
dummy provider as the default so we can verify something changed.
Should you have:
EXPECT_NE(d3, service->GetDefaultSearchProvider())
at the end?

http://codereview.chromium.org/8618001/diff/10004/chrome/browser/ui/views/fir...
chrome/browser/ui/views/first_run_search_engine_view_unittest.cc:28:
TemplateURL* d1 = new TemplateURL;
ASSERT_TRUE(service != NULL);

http://codereview.chromium.org/8618001/diff/10004/chrome/browser/ui/views/fir...
chrome/browser/ui/views/first_run_search_engine_view_unittest.cc:45:
service->GetTemplateURLs();
ASSERT_TRUE(!template_urls.empty())