no native implementation of Textfield.

chrome/browser/browser_main.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/browser/browser_main.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 137 matching lines @@
 #endif  // defined(OS_WIN)
 
 #if defined(OS_MACOSX)
 #include <Security/Security.h>
 #include "chrome/browser/ui/cocoa/install_from_dmg.h"
 #endif
 
 #if defined(TOOLKIT_VIEWS)
 #include "chrome/browser/views/chrome_views_delegate.h"
 #include "views/focus/accelerator_handler.h"
+#if defined(OS_LINUX)

[rjkroege, 2010/12/15 21:23:31]

Why OS_LINUX? We have been TOUCH_UI for the touch changes? This change ought to
work under win too? 

[oshima, 2010/12/16 01:15:19]

This currently does not compile on Windows due to the difference of keystroke
class. I'm going to enable this in the follow up CL.

+#include "views/controls/textfield/native_textfield_view.h"
+#endif
 #endif
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/cros/screen_lock_library.h"
 #include "chrome/browser/chromeos/customization_document.h"
 #include "chrome/browser/chromeos/external_metrics.h"
 #include "chrome/browser/chromeos/login/authenticator.h"
 #include "chrome/browser/chromeos/login/login_utils.h"
 #include "chrome/browser/chromeos/login/screen_locker.h"
@@ skipping 718 matching lines @@
   // It is important for this to happen before the first run dialog, as it
   // styles the dialog as well.
   gtk_util::InitRCStyles();
 #endif
 
 #if defined(TOOLKIT_VIEWS)
   // The delegate needs to be set before any UI is created so that windows
   // display the correct icon.
   if (!views::ViewsDelegate::views_delegate)
     views::ViewsDelegate::views_delegate = new ChromeViewsDelegate;
+
+#if defined(OS_LINUX)
+  if (parameters.command_line_.HasSwitch(switches::kEnableTextfieldView))

[beng (no_code_reviews), 2010/12/15 20:42:00]

This check should be done inside CreateWrapper, rather than here. If we add
every such experiment to browser_main it'll become unreadable.

[oshima, 2010/12/16 01:15:19]

I originally had it in CreateWrapper, but I moved here to put the switch in
chrome/common/chrome_switch.h. I put the switch back to
native_textfield_view.cc. Let me know if it's okay, or if you have alternative
suggestion.

+    views::NativeTextfieldView::SetEnableTextfieldView(true);
+#endif
+
 #endif
 
 #if defined(OS_WIN)
   gfx::PlatformFontWin::adjust_font_callback = &AdjustUIFont;
   gfx::PlatformFontWin::get_minimum_font_size_callback = &GetMinimumFontSize;
 
   // Init common control sex.
   INITCOMMONCONTROLSEX config;
   config.dwSize = sizeof(config);
   config.dwICC = ICC_WIN95_CLASSES;
@@ skipping 818 matching lines @@
 #if defined(OS_CHROMEOS)
   // To be precise, logout (browser shutdown) is not yet done, but the
   // remaining work is negligible, hence we say LogoutDone here.
   chromeos::BootTimesLoader::Get()->AddLogoutTimeMarker("LogoutDone",
                                                         false);
   chromeos::BootTimesLoader::Get()->WriteLogoutTimes();
 #endif
   TRACE_EVENT_END("BrowserMain", 0, 0);
   return result_code;
 }