Address post commit comments for CL 10828201


BUG=113688


Review URL: https://chromiumcodereview.appspot.com/10854079

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151020 0039d316-1c4b-4281-b951-d872f2087c98
parent 8a08a754
......@@ -154,7 +154,7 @@ class InternalPageInfoPopupView : public views::BubbleDelegateView {
////////////////////////////////////////////////////////////////////////////////
PopupHeaderView::PopupHeaderView(views::ButtonListener* close_button_listener)
: name_(NULL), status_(NULL) {
: name_(NULL), status_(NULL) {
views::GridLayout* layout = new views::GridLayout(this);
SetLayoutManager(layout);
......@@ -217,7 +217,7 @@ void PopupHeaderView::SetIdentityName(const string16& name) {
}
void PopupHeaderView::SetIdentityStatus(const string16& status,
SkColor text_color) {
SkColor text_color) {
status_->SetText(status);
status_->SetEnabledColor(text_color);
}
......@@ -236,8 +236,8 @@ InternalPageInfoPopupView::InternalPageInfoPopupView(views::View* anchor_view)
icon_view->SetImage(rb.GetImageSkiaNamed(IDR_PRODUCT_LOGO_26));
AddChildView(icon_view);
string16 text = l10n_util::GetStringUTF16(IDS_PAGE_INFO_INTERNAL_PAGE);
views::Label* label = new views::Label(text);
views::Label* label =
new views::Label(l10n_util::GetStringUTF16(IDS_PAGE_INFO_INTERNAL_PAGE));
label->SetMultiLine(true);
label->SetAllowCharacterBreak(true);
label->SetHorizontalAlignment(views::Label::ALIGN_LEFT);
......
......@@ -116,8 +116,7 @@ class WebsiteSettingsPopupView
// tab.
TabContents* tab_contents_;
// The presenter that controlls the Website Settings UI. |presenter_| is null
// if the popup is opened for a internal chrome url.
// The presenter that controlls the Website Settings UI.
scoped_ptr<WebsiteSettings> presenter_;
PopupHeaderView* header_; // Owned by views hierarchy.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment