Mac: Enable "Check Spelling While Typing" in Edit menu...

chrome/browser/renderer_host/render_view_host.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/renderer_host/render_view_host.h"
 
 #include <string>
 #include <utility>
 #include <vector>
 
@@ skipping 342 matching lines @@
                                                    int request_id) {
   CrossSiteRequestManager::GetInstance()->SetHasPendingCrossSiteRequest(
       process()->id(), routing_id(), has_pending_request);
   pending_request_id_ = request_id;
 }
 
 int RenderViewHost::GetPendingRequestId() {
   return pending_request_id_;
 }
 
+RenderViewHost::CommandState RenderViewHost::GetStateForCommand(
+    RenderViewCommand command) const {
+  switch (command) {

[brettw, 2011/01/13 05:59:55]

Any reason this isn't an if statement? It will be fewer lines and if we end up
with a bunch of them, we'll probably end up writing some kind of range check
rather than individually listing them all anyway.

[sail, 2011/01/13 23:38:15]

Done.

+    case RENDER_VIEW_COMMAND_TOGGLE_SPELL_CHECK:
+      break;
+    default:
+      LOG(DFATAL) << "Unknown command " << command;
+      break;
+  }
+
+  std::map<RenderViewCommand, CommandState>::const_iterator it =
+      command_states_.find(command);
+  if (it == command_states_.end()) {
+    CommandState state;
+    state.is_enabled = false;
+    state.checked_state = COMMAND_CHECKED_STATE_UNCHECKED;
+    return state;
+  }
+  return it->second;
+}
+
 void RenderViewHost::Stop() {
   Send(new ViewMsg_Stop(routing_id()));
 }
 
 void RenderViewHost::ReloadFrame() {
   Send(new ViewMsg_ReloadFrame(routing_id()));
 }
 
 bool RenderViewHost::PrintPages() {
   return Send(new ViewMsg_PrintPages(routing_id()));
@@ skipping 487 matching lines @@
     IPC_MESSAGE_HANDLER(ViewHostMsg_DetectedPhishingSite,
                         OnDetectedPhishingSite)
     IPC_MESSAGE_HANDLER(ViewHostMsg_ScriptEvalResponse, OnScriptEvalResponse)
     IPC_MESSAGE_HANDLER(ViewHostMsg_UpdateContentRestrictions,
                         OnUpdateContentRestrictions)
 #if defined(OS_MACOSX)
     IPC_MESSAGE_HANDLER(ViewHostMsg_ShowPopup, OnMsgShowPopup)
 #endif
     IPC_MESSAGE_HANDLER(ViewHostMsg_PagesReadyForPreview,
                         OnPagesReadyForPreview)
+    IPC_MESSAGE_HANDLER(ViewHostMsg_CommandStateChanged,
+                        OnCommandStateChanged)
     // Have the super handle all other messages.
     IPC_MESSAGE_UNHANDLED(handled = RenderWidgetHost::OnMessageReceived(msg))
   IPC_END_MESSAGE_MAP_EX()
 
   if (!msg_is_ok) {
     // The message had a handler, but its de-serialization failed.
     // Kill the renderer.
     UserMetrics::RecordAction(UserMetricsAction("BadMessageTerminate_RVH"));
     process()->ReceivedBadMessage();
   }
@@ skipping 1130 matching lines @@
   if (printer_query.get()) {
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         NewRunnableMethod(printer_query.get(),
                           &printing::PrinterQuery::StopWorker));
   }
 
   // Send the printingDone msg for now.
   Send(new ViewMsg_PrintingDone(routing_id(), params.document_cookie, true));
 }
+
+void RenderViewHost::OnCommandStateChanged(int command,
+                                           bool is_enabled,
+                                           int checked_state) {
+  RenderViewCommand render_view_command;
+  switch (command) {

[brettw, 2011/01/13 05:59:55]

I'd probably use an if for these switches as well (you could do the cast at the
end when you actually use it) and it would be almost half as long:

if (checked_state != COMMAND_CHECKED_STATE_UNCHECKED &&
    checked_state != COMMAND_CHECKED_STATE_CHECKED &&
    checked_state != COMMAND_CHECKED_STATE_MIXED) {
  NOTREACHED() << "Invalid checked state " << checked_state;
  return;
}

[sail, 2011/01/13 23:38:15]

Done.

+    case RENDER_VIEW_COMMAND_TOGGLE_SPELL_CHECK:
+      render_view_command = static_cast<RenderViewCommand>(command);
+      break;
+    default:
+      LOG(DFATAL) << "Unknown command " << command;
+      return;
+  }
+
+  CommandCheckedState command_checked_state;
+  switch (checked_state) {
+    case COMMAND_CHECKED_STATE_UNCHECKED:
+    case COMMAND_CHECKED_STATE_CHECKED:
+    case COMMAND_CHECKED_STATE_MIXED:
+      command_checked_state = static_cast<CommandCheckedState>(checked_state);
+      break;
+    default:
+      LOG(DFATAL) << "Invalid checked state " << checked_state;
+      return;
+  }
+
+  CommandState state;
+  state.is_enabled = is_enabled;
+  state.checked_state = command_checked_state;
+  command_states_[render_view_command] = state;
+}