UI polish for certificate viewer

chrome/browser/chromeos/frame/bubble_window_style.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_CHROMEOS_FRAME_BUBBLE_WINDOW_STYLE_H_
 #define CHROME_BROWSER_CHROMEOS_FRAME_BUBBLE_WINDOW_STYLE_H_
 #pragma once
 
 #include "third_party/skia/include/core/SkColor.h"
 
 namespace chromeos {
 
 // A class to share common definitions between GTK and Views implementations.
 enum BubbleWindowStyle {
   // Default style.
   STYLE_GENERIC = 0,
 
   // Show close button at the top right (left for RTL).
   // Deprecated, see BubbleWindow::Create().
   STYLE_XBAR = 1 << 0,
 
   // Show throbber for slow rendering.
   // Deprecated, see BubbleWindow::Create().
   STYLE_THROBBER = 1 << 1,
 
   // Content flush to edge, no padding.
-  STYLE_FLUSH = 1 << 2
+  STYLE_FLUSH = 1 << 2,
+
+  // Only for certificate viewer. Add close button at the top right and flush
+  // to edge. Refer to issue 102511 for detail.
+  STYLE_CERT = 1 << 3

[flackr, 2011/11/10 16:03:16]

This style probably isn't going to be specific to certificate viewer, and even
if it is it should have a generic name describing what it does. It's very
similar to STYLE_FLUSH except that you still add the padding to the title bar. I
think a name like STYLE_FLUSH_CONTENT would be better suited.

[bshe, 2011/11/21 16:09:37]

Move the style to chrome/brwoser/ui/dialog_style.h. Add a
"STYLE_XBAR_FLUSH_CONTENT". Since we also have a Xbar, it makes more sense to
add XBAR to the style name.

 };
 
 extern const SkColor kBubbleWindowBackgroundColor;
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_FRAME_BUBBLE_WINDOW_STYLE_H_