Chromium Code Reviews
Help | Chromium Project | Sign in
(28)

Issue 12094034: Add a link (to view Permissions) to the Extension details on chrome://extensions. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 3 months ago by Finnur (OOO - back Monday)
Modified:
2 years, 3 months ago
CC:
chromium-reviews, Aaron Boodman, arv, tfarina, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Add a link (to view Permissions) to the Extension details on chrome://extensions. BUG=170742 TEST=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=179916

Patch Set 1 #

Total comments: 6

Patch Set 2 : #

Messages

Total messages: 9 (0 generated)
Finnur (OOO - back Monday)
mpcomplete: chrome/browser/extensions/* chrome/browser/ui/views/extensions/extension_install_dialog_view.cc and generated_resources.grd jhawkins: chrome/browser/resources/* chrome/browser/ui/webui/extensions/* sky: OWNERS for extension_install_dialog_view.cc
2 years, 3 months ago (2013-01-29 13:52:18 UTC) #1
sky
LGTM https://codereview.chromium.org/12094034/diff/1/chrome/browser/ui/views/extensions/extension_install_dialog_view.cc File chrome/browser/ui/views/extensions/extension_install_dialog_view.cc (right): https://codereview.chromium.org/12094034/diff/1/chrome/browser/ui/views/extensions/extension_install_dialog_view.cc#newcode536 chrome/browser/ui/views/extensions/extension_install_dialog_view.cc:536: DCHECK((buttons & ui::DIALOG_BUTTON_CANCEL) > 0); DCHECK_GT
2 years, 3 months ago (2013-01-29 16:45:04 UTC) #2
James Hawkins
https://codereview.chromium.org/12094034/diff/1/chrome/browser/resources/extensions/extensions.html File chrome/browser/resources/extensions/extensions.html (right): https://codereview.chromium.org/12094034/diff/1/chrome/browser/resources/extensions/extensions.html#newcode108 chrome/browser/resources/extensions/extensions.html:108: href="#"></a> nit: Wrapped lines must have the closing tag ...
2 years, 3 months ago (2013-01-29 18:27:46 UTC) #3
Matt Perry
lgtm
2 years, 3 months ago (2013-01-29 22:45:20 UTC) #4
Finnur (OOO - back Monday)
Addressed comments. James, PTAL. https://codereview.chromium.org/12094034/diff/1/chrome/browser/ui/webui/extensions/extension_settings_handler.cc File chrome/browser/ui/webui/extensions/extension_settings_handler.cc (right): https://codereview.chromium.org/12094034/diff/1/chrome/browser/ui/webui/extensions/extension_settings_handler.cc#newcode510 chrome/browser/ui/webui/extensions/extension_settings_handler.cc:510: CHECK(false); Because this should never ...
2 years, 3 months ago (2013-01-30 18:27:32 UTC) #5
Finnur (OOO - back Monday)
jhawkins: ping.
2 years, 3 months ago (2013-01-31 11:45:08 UTC) #6
James Hawkins
lgtm
2 years, 3 months ago (2013-01-31 17:03:07 UTC) #7
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/finnur@chromium.org/12094034/10
2 years, 3 months ago (2013-01-31 17:04:00 UTC) #8
I haz the power (commit-bot)
2 years, 3 months ago (2013-01-31 19:05:10 UTC) #9
Message was sent while issue was closed.
Change committed as 179916
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld e0e3771