Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5327)

Unified Diff: chrome/browser/tab_contents/spellchecker_submenu_observer_hunspell.cc

Issue 8823009: Remove Hunspell on OS X - step 1 (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix review comments Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/tab_contents/spellchecker_submenu_observer_hunspell.cc
diff --git a/chrome/browser/tab_contents/spellchecker_submenu_observer.cc b/chrome/browser/tab_contents/spellchecker_submenu_observer_hunspell.cc
similarity index 91%
rename from chrome/browser/tab_contents/spellchecker_submenu_observer.cc
rename to chrome/browser/tab_contents/spellchecker_submenu_observer_hunspell.cc
index c54491a4efed97619464da04d6fd8a92be6c6648..5a750b25dd9b0a1f58d65dcdcdf76fa69997c0a9 100644
--- a/chrome/browser/tab_contents/spellchecker_submenu_observer.cc
+++ b/chrome/browser/tab_contents/spellchecker_submenu_observer_hunspell.cc
@@ -11,7 +11,6 @@
#include "chrome/browser/prefs/pref_member.h"
#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/spellchecker/spellcheck_host.h"
-#include "chrome/browser/spellchecker/spellchecker_platform_engine.h"
#include "chrome/browser/tab_contents/render_view_context_menu.h"
#include "chrome/browser/tab_contents/spelling_bubble_model.h"
#include "chrome/common/chrome_switches.h"
@@ -73,18 +72,6 @@ void SpellCheckerSubMenuObserver::InitMenu(const ContextMenuParams& params) {
l10n_util::GetStringUTF16(
IDS_CONTENT_CONTEXT_CHECK_SPELLING_OF_THIS_FIELD));
- // Add an item that shows the spelling panel if the platform spellchecker
- // supports it.
- if (SpellCheckerPlatform::SpellCheckerAvailable() &&
- SpellCheckerPlatform::SpellCheckerProvidesPanel()) {
- submenu_model_.AddCheckItem(
- IDC_SPELLPANEL_TOGGLE,
- l10n_util::GetStringUTF16(
- SpellCheckerPlatform::SpellingPanelVisible() ?
- IDS_CONTENT_CONTEXT_HIDE_SPELLING_PANEL :
- IDS_CONTENT_CONTEXT_SHOW_SPELLING_PANEL));
- }
-
#if defined(OS_WIN)
// If we have not integrated the spelling service, we show an "Ask Google for
// spelling suggestions" item. On the other hand, if we have integrated the
@@ -202,11 +189,6 @@ void SpellCheckerSubMenuObserver::ExecuteCommand(int command_id) {
rvh->Send(new SpellCheckMsg_ToggleSpellCheck(rvh->routing_id()));
break;
- case IDC_SPELLPANEL_TOGGLE:
- rvh->Send(new SpellCheckMsg_ToggleSpellPanel(
- rvh->routing_id(), SpellCheckerPlatform::SpellingPanelVisible()));
- break;
-
#if defined(OS_WIN)
case IDC_CONTENT_CONTEXT_SPELLING_TOGGLE:
// When a user chooses the "Ask Google for spelling suggestions" item, we

Powered by Google App Engine
This is Rietveld 408576698