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

Issue 455023002: Revert 288418 "Extend contextMenus API to support browser/page a..." (Closed)

Created:
6 years, 4 months ago by luken
Modified:
6 years, 4 months ago
Reviewers:
gpdavis
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 288418 "Extend contextMenus API to support browser/page a..." > Extend contextMenus API to support browser/page actions > > Screenshot: > http://i.imgur.com/RILqQqe.png > > BUG=234425 > > Review URL: https://codereview.chromium.org/359493005 TBR=gpdavis.chromium@gmail.com Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288422

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+66 lines, -362 lines) Patch
M trunk/src/chrome/browser/extensions/api/context_menus/context_menus_api_helpers.h View 3 chunks +0 lines, -18 lines 0 comments Download
M trunk/src/chrome/browser/extensions/api/context_menus/context_menus_api_helpers.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M trunk/src/chrome/browser/extensions/context_menu_matcher.h View 2 chunks +3 lines, -7 lines 0 comments Download
M trunk/src/chrome/browser/extensions/context_menu_matcher.cc View 7 chunks +15 lines, -50 lines 0 comments Download
M trunk/src/chrome/browser/extensions/extension_context_menu_model.h View 5 chunks +0 lines, -19 lines 0 comments Download
M trunk/src/chrome/browser/extensions/extension_context_menu_model.cc View 8 chunks +8 lines, -88 lines 0 comments Download
M trunk/src/chrome/browser/extensions/extension_context_menu_model_unittest.cc View 4 chunks +12 lines, -132 lines 0 comments Download
M trunk/src/chrome/browser/extensions/menu_manager.h View 2 chunks +2 lines, -4 lines 0 comments Download
M trunk/src/chrome/browser/extensions/menu_manager_factory.h View 1 chunk +0 lines, -3 lines 0 comments Download
M trunk/src/chrome/browser/extensions/menu_manager_factory.cc View 2 chunks +3 lines, -7 lines 0 comments Download
M trunk/src/chrome/browser/renderer_context_menu/render_view_context_menu.cc View 4 chunks +16 lines, -14 lines 0 comments Download
M trunk/src/chrome/browser/ui/app_list/app_context_menu.cc View 1 chunk +1 line, -4 lines 0 comments Download
M trunk/src/chrome/browser/ui/ash/launcher/launcher_context_menu.cc View 1 chunk +2 lines, -4 lines 0 comments Download
M trunk/src/chrome/common/extensions/api/context_menus.json View 3 chunks +2 lines, -8 lines 0 comments Download
M trunk/src/chrome/common/extensions/api/web_view_internal.json View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
luken
6 years, 4 months ago (2014-08-08 20:40:07 UTC) #1
luken
6 years, 4 months ago (2014-08-08 20:40:14 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r288422 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698