Reverting 27759.

webkit/glue/webview_impl.cc

 // Copyright (c) 2007-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.
 
 #include "config.h"
 #include "build/build_config.h"
 
 #include "base/compiler_specific.h"
 MSVC_PUSH_WARNING_LEVEL(0);
 #include "CSSStyleSelector.h"
@@ skipping 114 matching lines @@
 // text zoom to half and 3x the original text size.  These three values match
 // those in Apple's port in WebKit/WebKit/WebView/WebView.mm
 static const double kTextSizeMultiplierRatio = 1.2;
 static const double kMinTextSizeMultiplier = 0.5;
 static const double kMaxTextSizeMultiplier = 3.0;
 
 // The group name identifies a namespace of pages.  Page group is used on OSX
 // for some programs that use HTML views to display things that don't seem like
 // web pages to the user (so shouldn't have visited link coloring).  We only use
 // one page group.
-static const char* kPageGroupName = "default";
+// FIXME: This needs to go into the WebKit API implementation and be hidden
+//         from the API's users.
+const char* pageGroupName = "default";
 
 // Ensure that the WebKit::WebDragOperation enum values stay in sync with
 // the original WebCore::DragOperation constants.
 #define COMPILE_ASSERT_MATCHING_ENUM(webcore_name) \
    COMPILE_ASSERT(int(WebCore::webcore_name) == int(WebKit::Web##webcore_name),\
                   dummy##webcore_name)
 COMPILE_ASSERT_MATCHING_ENUM(DragOperationNone);
 COMPILE_ASSERT_MATCHING_ENUM(DragOperationCopy);
 COMPILE_ASSERT_MATCHING_ENUM(DragOperationLink);
 COMPILE_ASSERT_MATCHING_ENUM(DragOperationGeneric);
@@ skipping 209 matching lines @@
 
   // Restrict the access to the local file system
   // (see WebView.mm WebView::_commonInitializationWithFrameName).
   FrameLoader::setLocalLoadPolicy(
       FrameLoader::AllowLocalLoadsForLocalOnly);
 }
 
 // static
 void WebView::UpdateVisitedLinkState(uint64 link_hash) {
   WebCore::Page::visitedStateChanged(
-      WebCore::PageGroup::pageGroup(kPageGroupName), link_hash);
+      WebCore::PageGroup::pageGroup(pageGroupName), link_hash);
 }
 
 // static
 void WebView::ResetVisitedLinkState() {
   WebCore::Page::allVisitedStateChanged(
-      WebCore::PageGroup::pageGroup(kPageGroupName));
+      WebCore::PageGroup::pageGroup(pageGroupName));
 }
 
 WebViewImpl::WebViewImpl(WebViewDelegate* delegate)
     : delegate_(delegate),
       ALLOW_THIS_IN_INITIALIZER_LIST(back_forward_list_client_impl_(this)),
       ALLOW_THIS_IN_INITIALIZER_LIST(chrome_client_impl_(this)),
       ALLOW_THIS_IN_INITIALIZER_LIST(context_menu_client_impl_(this)),
       ALLOW_THIS_IN_INITIALIZER_LIST(drag_client_impl_(this)),
       ALLOW_THIS_IN_INITIALIZER_LIST(editor_client_impl_(this)),
       ALLOW_THIS_IN_INITIALIZER_LIST(inspector_client_impl_(this)),
@@ skipping 25 matching lines @@
 
   // the page will take ownership of the various clients
   page_.reset(new Page(&chrome_client_impl_,
                        &context_menu_client_impl_,
                        &editor_client_impl_,
                        &drag_client_impl_,
                        &inspector_client_impl_,
                        NULL));
 
   page_->backForwardList()->setClient(&back_forward_list_client_impl_);
-  page_->setGroupName(kPageGroupName);
+  page_->setGroupName(pageGroupName);
 }
 
 WebViewImpl::~WebViewImpl() {
   DCHECK(page_ == NULL);
 }
 
 RenderTheme* WebViewImpl::theme() const {
   return page_.get() ? page_->theme() : RenderTheme::defaultTheme().get();
 }
 
@@ skipping 1475 matching lines @@
   return spelling_panel_is_visible_;
 }
 
 void WebViewImpl::setTabsToLinks(bool enable) {
   tabs_to_links_ = enable;
 }
 
 bool WebViewImpl::tabsToLinks() const {
   return tabs_to_links_;
 }