Issue 331133002: Default to malware interstitial v3

Issue 331133002: Default to malware interstitial v3 (Closed)

Created:
6 years, 6 months ago by felt

Modified:
6 years, 6 months ago

Reviewers:
mattm

CC:
chromium-reviews

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

Project:
chromium

Visibility:
Public.

Default to malware interstitial v3 Changes Version3Enabled for the malware interstitial. Default is now v3 unless a flag or field trial say otherwise. BUG=381260 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278021

Created: 6 years, 6 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+10 lines, -3 lines)			Patch
	M	chrome/browser/safe_browsing/safe_browsing_blocking_page.cc	View	1 2	1 chunk	+10 lines, -3 lines	0 comments	Download

Total messages: 12 (0 generated)

felt

Please take a look. I plan to land this after 339503004 (the reporting checkbox) and ...

6 years, 6 months ago (2014-06-16 05:59:21 UTC) #1

commit-bot: I haz the power

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

6 years, 6 months ago (2014-06-17 19:03:08 UTC) #4

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/felt@chromium.org/331133002/20001

6 years, 6 months ago (2014-06-17 21:29:41 UTC) #6

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/felt@chromium.org/331133002/40001

6 years, 6 months ago (2014-06-17 22:59:21 UTC) #8

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/felt@chromium.org/331133002/40001

6 years, 6 months ago (2014-06-18 05:10:29 UTC) #11

Message was sent while issue was closed.

Change committed as 278021