Use cocoa controls for HTML select popups in test_shell

webkit/tools/test_shell/test_webview_delegate_win.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file contains the implementation of TestWebViewDelegate, which serves
 // as the WebViewDelegate for the TestShellWebHost.  The host is expected to
 // have initialized a MessageLoop before these methods are called.
 
 #include "webkit/tools/test_shell/test_webview_delegate.h"
 
@@ skipping 60 matching lines @@
 void TestWebViewDelegate::Show(WebWidget* webwidget, WindowOpenDisposition) {
   if (webwidget == shell_->webView()) {
     ShowWindow(shell_->mainWnd(), SW_SHOW);
     UpdateWindow(shell_->mainWnd());
   } else if (webwidget == shell_->popup()) {
     ShowWindow(shell_->popupWnd(), SW_SHOW);
     UpdateWindow(shell_->popupWnd());
   }
 }
 
+void TestWebViewDelegate::ShowWithItems(WebWidget* webwidget,
+                                        const gfx::Rect& bounds,
+                                        int item_height,
+                                        int selected_index,
+                                        const std::vector<MenuItem>& items) {
+  NOTIMPLEMENTED();

[darin (slow to review), 2009/04/01 18:02:50]

this macro suggests that this method should one day be implemented, but is that
really what you intend?  perhaps NOTREACHED is better since we never intend to
implement this method, right?  same for the GTK case.

+}
+
 void TestWebViewDelegate::CloseWidgetSoon(WebWidget* webwidget) {
   if (webwidget == shell_->webView()) {
     PostMessage(shell_->mainWnd(), WM_CLOSE, 0, 0);
   } else if (webwidget == shell_->popup()) {
     shell_->ClosePopup();
   }
 }
 
 void TestWebViewDelegate::SetCursor(WebWidget* webwidget,
                                     const WebCursor& cursor) {
@@ skipping 90 matching lines @@
 
 void TestWebViewDelegate::SetPageTitle(const std::wstring& title) {
   // The Windows test shell, pre-refactoring, ignored this.  *shrug*
 }
 
 void TestWebViewDelegate::SetAddressBarURL(const GURL& url) {
   std::wstring url_string = UTF8ToWide(url.spec());
   SendMessage(shell_->editWnd(), WM_SETTEXT, 0,
               reinterpret_cast<LPARAM>(url_string.c_str()));
 }