Issue 9121064: Cleanup: Remove more uses of Browser::tabstrip_model().

Issue 9121064: Cleanup: Remove more uses of Browser::tabstrip_model(). (Closed)

Created:
8 years, 11 months ago by Lei Zhang

Modified:
8 years, 10 months ago

Reviewers:
Robert Sesek, Elliot Glaysher, Ben Goodger (Google), akalin

CC:
chromium-reviews, robertshield, kkania

Base URL:
svn://chrome-svn/chrome/trunk/src/

Visibility:
Public.

Cleanup: Remove more uses of Browser::tabstrip_model(). BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=120125

Created: 8 years, 10 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+23 lines, -26 lines)			Patch
M	chrome/browser/automation/testing_automation_provider.cc	View	1 2	5 chunks	+7 lines, -10 lines	0 comments	Download
M	chrome/browser/sessions/session_restore.cc	View	1 2	2 chunks	+2 lines, -2 lines	0 comments	Download
M	chrome/browser/sessions/session_service.cc	View	1 2	2 chunks	+2 lines, -2 lines	0 comments	Download
M	chrome/browser/ui/cocoa/browser_window_controller.mm	View	1 2	3 chunks	+3 lines, -3 lines	0 comments	Download
M	chrome/browser/ui/gtk/browser_window_gtk.cc	View	1 2	3 chunks	+4 lines, -4 lines	0 comments	Download
M	chrome/browser/ui/sync/browser_synced_window_delegate.cc	View	1 2	2 chunks	+4 lines, -4 lines	0 comments	Download
M	chrome/browser/ui/views/frame/browser_view.cc	View	1 2	1 chunk	+1 line, -1 line	0 comments	Download

Total messages: 13 (0 generated)

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/9121064/6001

8 years, 10 months ago (2012-01-31 21:15:43 UTC) #4

commit-bot: I haz the power

Presubmit check for 9121064-6001 failed and returned exit status 1. Running presubmit commit checks ...

8 years, 10 months ago (2012-01-31 21:15:50 UTC) #5

Lei Zhang

OWNERs stamp collection time: erg: gtk akalin: sync rsesek: mac

8 years, 10 months ago (2012-01-31 22:42:34 UTC) #6

Elliot Glaysher

On 2012/01/31 22:42:34, Lei Zhang wrote: > OWNERs stamp collection time: > > erg: gtk ...

8 years, 10 months ago (2012-01-31 22:43:28 UTC) #7

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/9121064/6001

8 years, 10 months ago (2012-01-31 23:24:25 UTC) #10

commit-bot: I haz the power

Try job failure for 9121064-6001 (retry) on mac_rel for step "browser_tests". It's a second try, ...

8 years, 10 months ago (2012-02-01 01:12:14 UTC) #11

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/9121064/14001

8 years, 10 months ago (2012-02-01 21:51:29 UTC) #12

Change committed as 120125