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

Unified Diff: chrome/browser/extensions/api/context_menu/context_menu_api.cc

Issue 10907151: Extensions Docs Server: Enum values do not show up if enum is a type (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rtcPrivate fix Created 8 years, 3 months 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/extensions/api/context_menu/context_menu_api.cc
diff --git a/chrome/browser/extensions/api/context_menu/context_menu_api.cc b/chrome/browser/extensions/api/context_menu/context_menu_api.cc
index a87a3491ec37919f936efeaf2b67368fb56ab5ec..102d5ee4a359cc9b55d99a1de8e2caf6bd190bb3 100644
--- a/chrome/browser/extensions/api/context_menu/context_menu_api.cc
+++ b/chrome/browser/extensions/api/context_menu/context_menu_api.cc
@@ -75,6 +75,8 @@ extensions::MenuItem::ContextList GetContexts(
case PropertyWithEnumT::CONTEXTS_ELEMENT_FRAME:
contexts.Add(extensions::MenuItem::FRAME);
break;
+ case PropertyWithEnumT::CONTEXTS_ELEMENT_NONE:
+ NOTREACHED();
}
}
return contexts;
@@ -243,6 +245,8 @@ bool UpdateContextMenuFunction::RunImpl() {
case Update::Params::ID_INTEGER:
item_id.uid = *params->id_integer;
break;
+ case Update::Params::ID_NONE:
+ NOTREACHED();
}
ExtensionService* service = profile()->GetExtensionService();
@@ -344,6 +348,9 @@ bool RemoveContextMenuFunction::RunImpl() {
break;
case Remove::Params::MENU_ITEM_ID_INTEGER:
id.uid = *params->menu_item_id_integer;
+ break;
+ case Remove::Params::MENU_ITEM_ID_NONE:
+ NOTREACHED();
}
MenuItem* item = manager->GetItemById(id);

Powered by Google App Engine
This is Rietveld 408576698