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

Unified Diff: chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm

Issue 2385613002: Mac Page Info constants: Use common Chrome comment heading style. (Closed)
Patch Set: Created 4 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
diff --git a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
index bb2e0e71dec707f478446009495a27c780a0d50c..a6a2128724f92a3941a32e8b7fbfedf41b6704d3 100644
--- a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
+++ b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
@@ -58,7 +58,7 @@ using ChosenObjectDeleteCallback =
namespace {
-/**************** General ****************/
+// General ---------------------------------------------------------------------
// The default width of the window, in view coordinates. It may be larger to
// fit the content.
@@ -72,12 +72,12 @@ const CGFloat kSectionHorizontalPadding = 16;
// align with other text.
const CGFloat kLinkButtonXAdjustment = 1;
-/**************** Security Section ****************/
+// Security Section ------------------------------------------------------------
// Spacing between security summary, security details, and cert decisions text.
const CGFloat kSecurityParagraphSpacing = 12;
-/**************** Site Settings Section ****************/
+// Site Settings Section -------------------------------------------------------
// Square size of the permission images.
const CGFloat kPermissionImageSize = 16;
@@ -99,7 +99,7 @@ const CGFloat kPermissionIconYAdjustment = 1;
// permission label.
const CGFloat kPermissionPopupButtonYAdjustment = 3;
-/**************** Internal Page Bubble ****************/
+// Internal Page Bubble --------------------------------------------------------
// Padding between the window frame and content for the internal page bubble.
const CGFloat kInternalPageFramePadding = 10;
@@ -107,7 +107,7 @@ const CGFloat kInternalPageFramePadding = 10;
// Spacing between the image and text for internal pages.
const CGFloat kInternalPageImageSpacing = 10;
-/********************************/
+// -----------------------------------------------------------------------------
// NOTE: This assumes that there will never be more than one website settings
// popup shown, and that the one that is shown is associated with the current
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698