Chromium Code Reviews| Index: chrome/browser/ui/browser.cc |
| =================================================================== |
| --- chrome/browser/ui/browser.cc (revision 96104) |
| +++ chrome/browser/ui/browser.cc (working copy) |
| @@ -1116,7 +1116,12 @@ |
| browser::NavigateParams params(this, url, transition); |
| params.disposition = NEW_FOREGROUND_TAB; |
| browser::Navigate(¶ms); |
| - return params.target_contents; |
| + TabContentsWrapper* target_contents_wrapper = params.target_contents; |
| + NotificationService::current()->Notify( |
|
Peter Kasting
2011/08/12 17:19:45
Is this necessary? It seems like this ought to be
jianli
2011/08/16 20:55:57
Changed to call Notify from Navigate(), as suggest
|
| + content::NOTIFICATION_TAB_ADDED, |
| + Source<TabContentsDelegate>(params.browser), |
| + Details<TabContents>(target_contents_wrapper->tab_contents())); |
| + return target_contents_wrapper; |
| } |
| TabContents* Browser::AddTab(TabContentsWrapper* tab_contents, |
| @@ -3508,6 +3513,11 @@ |
| return GetJavaScriptDialogCreatorInstance(); |
| } |
| +void Browser::UpdatePreferredSize(TabContents* source, |
| + const gfx::Size& pref_size) { |
| + window_->UpdatePreferredSize(source, pref_size); |
| +} |
| + |
| /////////////////////////////////////////////////////////////////////////////// |
| // Browser, TabContentsWrapperDelegate implementation: |