Reverting 27711.

webkit/glue/webview_impl.cc

Index: webkit/glue/webview_impl.cc
===================================================================
--- webkit/glue/webview_impl.cc	(revision 27750)
+++ webkit/glue/webview_impl.cc	(working copy)
@@ -338,7 +338,7 @@
   return instance;
 }
 
-void WebViewImpl::InitializeMainFrame(WebFrameClient* frame_client) {
+void WebViewImpl::initializeMainFrame(WebFrameClient* frame_client) {
   // NOTE: The WebFrameImpl takes a reference to itself within InitMainFrame
   // and releases that reference once the corresponding Frame is destroyed.
   scoped_refptr<WebFrameImpl> main_frame = new WebFrameImpl(frame_client);
@@ -370,7 +370,6 @@
       WebCore::PageGroup::pageGroup(kPageGroupName));
 }
 
-
 WebViewImpl::WebViewImpl(WebViewDelegate* delegate)
     : delegate_(delegate),
       ALLOW_THIS_IN_INITIALIZER_LIST(back_forward_list_client_impl_(this)),
@@ -425,7 +424,12 @@
   return page_.get() ? page_->theme() : RenderTheme::defaultTheme().get();
 }
 
-void WebViewImpl::SetTabKeyCyclesThroughElements(bool value) {
+bool WebViewImpl::tabKeyCyclesThroughElements() const {
+  ASSERT(page_.get());
+  return page_->tabKeyCyclesThroughElements();
+}
+
+void WebViewImpl::setTabKeyCyclesThroughElements(bool value) {
   if (page_ != NULL) {
     page_->setTabKeyCyclesThroughElements(value);
   }
@@ -1302,12 +1306,11 @@
     relative_to_frame = mainFrame();
   Frame* frame = static_cast<WebFrameImpl*>(relative_to_frame)->frame();
   frame = frame->tree()->find(name_str);
-  return frame ? WebFrameImpl::FromFrame(frame) : NULL;
+  return WebFrameImpl::FromFrame(frame);
 }
 
 WebFrame* WebViewImpl::focusedFrame() {
-  Frame* frame = GetFocusedWebCoreFrame();
-  return frame ? WebFrameImpl::FromFrame(frame) : NULL;
+  return WebFrameImpl::FromFrame(GetFocusedWebCoreFrame());
 }
 
 void WebViewImpl::setFocusedFrame(WebFrame* frame) {
@@ -1607,30 +1610,6 @@
   return delegate_;
 }
 
-WebFrame* WebViewImpl::GetPreviousFrameBefore(WebFrame* frame, bool wrap) {
-  WebFrameImpl* frame_impl = static_cast<WebFrameImpl*>(frame);
-  WebCore::Frame* previous =
-      frame_impl->frame()->tree()->traversePreviousWithWrap(wrap);
-  return previous ? WebFrameImpl::FromFrame(previous) : NULL;
-}
-
-WebFrame* WebViewImpl::GetNextFrameAfter(WebFrame* frame, bool wrap) {
-  WebFrameImpl* frame_impl = static_cast<WebFrameImpl*>(frame);
-  WebCore::Frame* next =
-      frame_impl->frame()->tree()->traverseNextWithWrap(wrap);
-  return next ? WebFrameImpl::FromFrame(next) : NULL;
-}
-
-// TODO(darin): these navigation methods should be killed
-
-void WebViewImpl::StopLoading() {
-  main_frame()->stopLoading();
-}
-
-void WebViewImpl::SetBackForwardListSize(int size) {
-  page_->backForwardList()->setCapacity(size);
-}
-
 const std::wstring& WebViewImpl::GetInspectorSettings() const {
   return inspector_settings_;
 }
@@ -1639,11 +1618,7 @@
   inspector_settings_ = settings;
 }
 
-void WebViewImpl::ShowJavaScriptConsole() {
-  page_->inspectorController()->showPanel(InspectorController::ConsolePanel);
-}
-
-bool WebViewImpl::SetDropEffect(bool accept) {
+bool WebViewImpl::setDropEffect(bool accept) {
   if (drag_target_dispatch_) {
     drop_effect_ = accept ? DROP_EFFECT_COPY : DROP_EFFECT_NONE;
     return true;
@@ -1721,7 +1696,7 @@
   return devtools_agent_.get();
 }
 
-void WebViewImpl::SetIsTransparent(bool is_transparent) {
+void WebViewImpl::setIsTransparent(bool is_transparent) {
   // Set any existing frames to be transparent.
   WebCore::Frame* frame = page_->mainFrame();
   while (frame) {
@@ -1733,7 +1708,7 @@
   is_transparent_ = is_transparent;
 }
 
-bool WebViewImpl::GetIsTransparent() const {
+bool WebViewImpl::isTransparent() const {
   return is_transparent_;
 }
 
@@ -1768,12 +1743,12 @@
   }
 }
 
-void WebViewImpl::SetActive(bool active) {
+void WebViewImpl::setIsActive(bool active) {
   if (page() && page()->focusController())
     page()->focusController()->setActive(active);
 }
 
-bool WebViewImpl::IsActive() {
+bool WebViewImpl::isActive() const {
   return (page() && page()->focusController())
       ? page()->focusController()->isActive()
       : false;
@@ -1897,10 +1872,10 @@
   return spelling_panel_is_visible_;
 }
 
-void WebViewImpl::SetTabsToLinks(bool enable) {
+void WebViewImpl::setTabsToLinks(bool enable) {
   tabs_to_links_ = enable;
 }
 
-bool WebViewImpl::GetTabsToLinks() const {
+bool WebViewImpl::tabsToLinks() const {
   return tabs_to_links_;
 }