Implement WebKitTabToLinksPreferenceKey for the test_shell....

webkit/glue/webview.h

 // Copyright (c) 2009 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.
 
 #ifndef WEBKIT_GLUE_WEBVIEW_H_
 #define WEBKIT_GLUE_WEBVIEW_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 230 matching lines @@
   virtual WebDevToolsAgent* GetWebDevToolsAgent() = 0;
 
   // Makes the webview transparent. Useful if you want to have some custom
   // background behind it.
   virtual void SetIsTransparent(bool is_transparent) = 0;
   virtual bool GetIsTransparent() const = 0;
 
   virtual void SetSpellingPanelVisibility(bool is_visible) = 0;
   virtual bool GetSpellingPanelVisibility() = 0;
 
+  virtual void SetTabsToLinks(bool enable) = 0;
+  virtual bool GetTabsToLinks() const = 0;

[darin (slow to review), 2009/09/17 19:24:54]

hmm... only the setter is really needed on the interface.  you could also
implement SetTabsToLinks by calling a method on the ChromeClientImpl to set a
member variable of ChromeClientImpl.

That said, I see that you are just following the pattern of
Set/GetSpellingPanelVisibility and Set/GetIsTransparent.

I'm OK with this change on that basis.  I might reconsider all of these methods
once I get around to moving WebView into webkit/api/public.

+
   // Performs an action from a context menu for the node at the given
   // location.
   virtual void MediaPlayerActionAt(int x,
                                    int y,
                                    const MediaPlayerAction& action) = 0;
 
   // Updates the WebView's active state (i.e., control tints).
   virtual void SetActive(bool active) = 0;
 
   // Gets the WebView's active state (i.e., state of control tints).
   virtual bool IsActive() = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(WebView);
 };
 
 #endif  // WEBKIT_GLUE_WEBVIEW_H_