Remove redundant separator at end of the spelling correction context menu

chrome/browser/renderer_context_menu/render_view_context_menu.cc

Index: chrome/browser/renderer_context_menu/render_view_context_menu.cc
diff --git a/chrome/browser/renderer_context_menu/render_view_context_menu.cc b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
index f86db71d8c2bd0bb784b87ddb39ee6aa4ccf0db2..715fd2909771a1c148a7fcad23f6f44c6b952777 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
@@ -1410,8 +1410,6 @@ void RenderViewContextMenu::AppendEditableItems() {
     menu_model_.AddItemWithStringId(IDC_CONTENT_CONTEXT_SELECTALL,
                                     IDS_CONTENT_CONTEXT_SELECTALL);
   }
-
-  menu_model_.AddSeparator(ui::NORMAL_SEPARATOR);

[lazyboy, 2017/02/28 00:29:02]

This is prone to further bugs I think:
If group items that come after AppendEditableItems() do not prepend a separator
then we will have two groups unintentionally merged?

Instead we can remove a separator at the end of ::InitMenu(), sth like:
if (menu_model_.GetItemCount() > 0 &&
menu_model_.GetTypeAt(menu_model_.GetItemCount()-1) == TYPE_SEPARATOR)
  menu_model_.RemoveItemAt(...)

WDYT?

[edwardjung, 2017/02/28 01:01:06]

Done.

 }
 
 void RenderViewContextMenu::AppendLanguageSettings() {