Issue 9695043: Guard off two more places for ENABLE_NOTIFICATIONS

Issue 9695043: Guard off two more places for ENABLE_NOTIFICATIONS (Closed)

Created:
8 years, 9 months ago by Yaron

Modified:
8 years, 9 months ago

Reviewers:
Jay Civelli, Elliot Glaysher

CC:
chromium-reviews

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

Created: 8 years, 9 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+7 lines, -0 lines)			Patch
	M	chrome/browser/profiles/profile_dependency_manager.cc	View		1 chunk	+2 lines, -0 lines	0 comments	Download
	M	chrome/test/base/testing_browser_process.cc	View		1 chunk	+5 lines, -0 lines	0 comments	Download

Total messages: 8 (0 generated)

Yaron

On 2012/03/13 16:05:30, Jay Civelli wrote: > lgtm erg: can you please look at chrome/browser/profiles

8 years, 9 months ago (2012-03-15 13:54:00 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yfriedman@chromium.org/9695043/1

8 years, 9 months ago (2012-03-15 16:54:53 UTC) #5

commit-bot: I haz the power

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

8 years, 9 months ago (2012-03-15 20:15:57 UTC) #6

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yfriedman@chromium.org/9695043/1

8 years, 9 months ago (2012-03-15 23:11:07 UTC) #7

Change committed as 127089