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

Side by Side Diff: chrome/browser/ui/views/translate/translate_bubble_view.cc

Issue 1759453002: Convert location bar bubble delegates to bubble dialog delegates (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix compile errs Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "chrome/browser/ui/views/translate/translate_bubble_view.h" 5 #include "chrome/browser/ui/views/translate/translate_bubble_view.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <algorithm> 8 #include <algorithm>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 scoped_ptr<translate::TranslateUIDelegate> ui_delegate( 113 scoped_ptr<translate::TranslateUIDelegate> ui_delegate(
114 new translate::TranslateUIDelegate( 114 new translate::TranslateUIDelegate(
115 ChromeTranslateClient::GetManagerFromWebContents(web_contents) 115 ChromeTranslateClient::GetManagerFromWebContents(web_contents)
116 ->GetWeakPtr(), 116 ->GetWeakPtr(),
117 source_language, 117 source_language,
118 target_language)); 118 target_language));
119 scoped_ptr<TranslateBubbleModel> model( 119 scoped_ptr<TranslateBubbleModel> model(
120 new TranslateBubbleModelImpl(step, std::move(ui_delegate))); 120 new TranslateBubbleModelImpl(step, std::move(ui_delegate)));
121 TranslateBubbleView* view = new TranslateBubbleView( 121 TranslateBubbleView* view = new TranslateBubbleView(
122 anchor_view, std::move(model), error_type, web_contents); 122 anchor_view, std::move(model), error_type, web_contents);
123 views::Widget* bubble_widget = views::BubbleDelegateView::CreateBubble(view); 123 views::Widget* bubble_widget =
124 views::BubbleDialogDelegateView::CreateBubble(view);
124 view->ShowForReason(reason); 125 view->ShowForReason(reason);
125 return bubble_widget; 126 return bubble_widget;
126 } 127 }
127 128
128 // static 129 // static
129 void TranslateBubbleView::CloseBubble() { 130 void TranslateBubbleView::CloseCurrentBubble() {
130 if (!translate_bubble_view_) 131 if (!translate_bubble_view_)
131 return; 132 return;
132 133
133 translate_bubble_view_->GetWidget()->Close(); 134 translate_bubble_view_->CloseBubble();
msw 2016/03/03 23:01:40 optional nit: invert conditional and nest, like ot
Evan Stade 2016/03/07 18:36:38 Done.
134 } 135 }
135 136
136 // static 137 // static
137 TranslateBubbleView* TranslateBubbleView::GetCurrentBubble() { 138 TranslateBubbleView* TranslateBubbleView::GetCurrentBubble() {
138 return translate_bubble_view_; 139 return translate_bubble_view_;
139 } 140 }
140 141
141 void TranslateBubbleView::Init() { 142 void TranslateBubbleView::Init() {
142 SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 143 SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical,
143 0, 0, 0)); 144 0, 0, 0));
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
206 case TranslateBubbleModel::VIEW_STATE_ERROR: 207 case TranslateBubbleModel::VIEW_STATE_ERROR:
207 break; 208 break;
208 case TranslateBubbleModel::VIEW_STATE_ADVANCED: { 209 case TranslateBubbleModel::VIEW_STATE_ADVANCED: {
209 if (accelerator.key_code() == ui::VKEY_RETURN) { 210 if (accelerator.key_code() == ui::VKEY_RETURN) {
210 HandleButtonPressed(BUTTON_ID_DONE); 211 HandleButtonPressed(BUTTON_ID_DONE);
211 return true; 212 return true;
212 } 213 }
213 break; 214 break;
214 } 215 }
215 } 216 }
216 return BubbleDelegateView::AcceleratorPressed(accelerator); 217 return BubbleDialogDelegateView::AcceleratorPressed(accelerator);
217 } 218 }
218 219
219 gfx::Size TranslateBubbleView::GetPreferredSize() const { 220 gfx::Size TranslateBubbleView::GetPreferredSize() const {
220 int width = 0; 221 int width = 0;
221 for (int i = 0; i < child_count(); i++) { 222 for (int i = 0; i < child_count(); i++) {
222 const views::View* child = child_at(i); 223 const views::View* child = child_at(i);
223 width = std::max(width, child->GetPreferredSize().width()); 224 width = std::max(width, child->GetPreferredSize().width());
224 } 225 }
225 int height = GetCurrentView()->GetPreferredSize().height(); 226 int height = GetCurrentView()->GetPreferredSize().height();
226 return gfx::Size(width, height); 227 return gfx::Size(width, height);
(...skipping 526 matching lines...) Expand 10 before | Expand all | Expand 10 after
753 base::string16 label; 754 base::string16 label;
754 if (model_->IsPageTranslatedInCurrentLanguages()) 755 if (model_->IsPageTranslatedInCurrentLanguages())
755 label = l10n_util::GetStringUTF16(IDS_DONE); 756 label = l10n_util::GetStringUTF16(IDS_DONE);
756 else 757 else
757 label = l10n_util::GetStringUTF16(IDS_TRANSLATE_BUBBLE_ACCEPT); 758 label = l10n_util::GetStringUTF16(IDS_TRANSLATE_BUBBLE_ACCEPT);
758 advanced_done_button_->SetText(label); 759 advanced_done_button_->SetText(label);
759 advanced_done_button_->SizeToPreferredSize(); 760 advanced_done_button_->SizeToPreferredSize();
760 if (advanced_view_) 761 if (advanced_view_)
761 advanced_view_->Layout(); 762 advanced_view_->Layout();
762 } 763 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698