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

Issue 10789053: Bring up a content settings icon for ungestured registerProtocolHandler call. (Closed)

Created:
8 years, 5 months ago by Scott Hess - ex-Googler
Modified:
8 years, 5 months ago
Reviewers:
benwells, Greg Billock
CC:
chromium-reviews, markusheintz_
Visibility:
Public.

Description

Bring up a content settings icon for ungestured registerProtocolHandler call. Reverted by shess@chromium.org, turned out not to be related to the problem. Revert CL: https://chromiumcodereview.appspot.com/10782038/ Original CL: https://chromiumcodereview.appspot.com/10584042 R=benwells@chromium.org BUG=122020 TEST=None TBR=gbillock@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=147135

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+716 lines, -122 lines) Patch
A + chrome/app/nibs/ContentProtocolHandlers.xib View 21 chunks +213 lines, -74 lines 0 comments Download
M chrome/browser/content_settings/content_settings_default_provider.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/content_settings/content_settings_policy_provider.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/content_settings/content_settings_utils.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/content_settings/tab_specific_content_settings.h View 3 chunks +51 lines, -0 lines 0 comments Download
M chrome/browser/content_settings/tab_specific_content_settings.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/ui/browser.h View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/browser.cc View 4 chunks +20 lines, -6 lines 0 comments Download
M chrome/browser/ui/browser_content_setting_bubble_model_delegate.cc View 2 chunks +9 lines, -2 lines 0 comments Download
M chrome/browser/ui/cocoa/content_settings/content_setting_bubble_cocoa.mm View 5 chunks +11 lines, -19 lines 0 comments Download
M chrome/browser/ui/content_settings/content_setting_bubble_model.cc View 4 chunks +179 lines, -14 lines 0 comments Download
M chrome/browser/ui/content_settings/content_setting_bubble_model_unittest.cc View 4 chunks +148 lines, -0 lines 0 comments Download
M chrome/browser/ui/content_settings/content_setting_image_model.cc View 3 chunks +30 lines, -0 lines 0 comments Download
M chrome/browser/ui/content_settings/content_setting_image_model_unittest.cc View 2 chunks +17 lines, -0 lines 0 comments Download
M chrome/browser/ui/gtk/gtk_theme_service.cc View 2 chunks +3 lines, -1 line 0 comments Download
M chrome/browser/ui/libgtk2ui/gtk2_ui.cc View 2 chunks +3 lines, -1 line 0 comments Download
M chrome/browser/ui/views/external_tab_container_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/options2/content_settings_handler.cc View 3 chunks +8 lines, -0 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/chrome_dll.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/content_settings_types.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/test/base/testing_profile.h View 2 chunks +5 lines, -0 lines 0 comments Download
M chrome/test/base/testing_profile.cc View 2 chunks +6 lines, -2 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698