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

Side by Side Diff: chrome/browser/ui/views/web_intent_picker_view.cc

Issue 9005015: Coverity: Initialize member variables. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Build fix. Created 9 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <algorithm> 5 #include <algorithm>
6 #include <vector> 6 #include <vector>
7 7
8 #include "chrome/browser/ui/browser.h" 8 #include "chrome/browser/ui/browser.h"
9 #include "chrome/browser/ui/intents/web_intent_picker.h" 9 #include "chrome/browser/ui/intents/web_intent_picker.h"
10 #include "chrome/browser/ui/intents/web_intent_picker_delegate.h" 10 #include "chrome/browser/ui/intents/web_intent_picker_delegate.h"
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 browser::CreateViewsBubble(bubble_delegate); 141 browser::CreateViewsBubble(bubble_delegate);
142 bubble_delegate->Show(); 142 bubble_delegate->Show();
143 return bubble_delegate; 143 return bubble_delegate;
144 } 144 }
145 145
146 WebIntentPickerView::WebIntentPickerView(views::View* anchor_view, 146 WebIntentPickerView::WebIntentPickerView(views::View* anchor_view,
147 TabContentsWrapper* wrapper, 147 TabContentsWrapper* wrapper,
148 WebIntentPickerDelegate* delegate) 148 WebIntentPickerDelegate* delegate)
149 : BubbleDelegateView(anchor_view, views::BubbleBorder::TOP_LEFT), 149 : BubbleDelegateView(anchor_view, views::BubbleBorder::TOP_LEFT),
150 delegate_(delegate), 150 delegate_(delegate),
151 button_hbox_(NULL) {} 151 button_hbox_(NULL),
152 plus_button_(NULL) {
153 }
152 154
153 WebIntentPickerView::~WebIntentPickerView() { 155 WebIntentPickerView::~WebIntentPickerView() {
154 } 156 }
155 157
156 void WebIntentPickerView::ButtonPressed(views::Button* sender, 158 void WebIntentPickerView::ButtonPressed(views::Button* sender,
157 const views::Event& event) { 159 const views::Event& event) {
158 // TODO(binji) When we support the plus button, pressing it should forward the 160 // TODO(binji) When we support the plus button, pressing it should forward the
159 // user to services in the Chrome Web Store that provide this intent. 161 // user to services in the Chrome Web Store that provide this intent.
160 if (sender == plus_button_) 162 if (sender == plus_button_)
161 return; 163 return;
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
251 views::Label* bottom_label = new views::Label( 253 views::Label* bottom_label = new views::Label(
252 l10n_util::GetStringUTF16(IDS_FIND_MORE_INTENT_HANDLER_MESSAGE)); 254 l10n_util::GetStringUTF16(IDS_FIND_MORE_INTENT_HANDLER_MESSAGE));
253 bottom_label->SetMultiLine(true); 255 bottom_label->SetMultiLine(true);
254 bottom_label->SetHorizontalAlignment(views::Label::ALIGN_LEFT); 256 bottom_label->SetHorizontalAlignment(views::Label::ALIGN_LEFT);
255 bottom_label->SetFont( 257 bottom_label->SetFont(
256 bottom_label->font().DeriveFont(kWebStoreLabelFontDelta)); 258 bottom_label->font().DeriveFont(kWebStoreLabelFontDelta));
257 main_content->AddChildView(bottom_label); 259 main_content->AddChildView(bottom_label);
258 260
259 AddChildView(main_content); 261 AddChildView(main_content);
260 } 262 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/sad_tab_view.cc ('k') | content/browser/renderer_host/render_message_filter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698