Issue 511773002: Cleanup: Remove some unneeded SSE2 checks and unused code. (try 2)

Issue 511773002: Cleanup: Remove some unneeded SSE2 checks and unused code. (try 2) (Closed)

Created:
6 years, 3 months ago by Lei Zhang

Modified:
6 years, 3 months ago

Reviewers:
cpu_(ooo_6.6-7.5), Stephen White

CC:
chromium-reviews, DaleCurtis, rileya (GONE FROM CHROMIUM)

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Project:
chromium

Visibility:
Public.

Cleanup: Remove some unneeded SSE2 checks and unused code. (try 2) This is a resubmit of https://codereview.chromium.org/459603003 without the media/ changes. BUG=349320 TBR=cpu@chromium.org,senorblanco@chromium.org Committed: https://crrev.com/f2e5463e1d4f50bad11ee48056d6cb6037e20d1c Cr-Commit-Position: refs/heads/master@{#292121}

Created: 6 years, 3 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+8 lines, -50 lines)			Patch
M	chrome/app/chromium_strings.grd	View	1 chunk	+0 lines, -6 lines	0 comments	Download
M	chrome/app/google_chrome_strings.grd	View	1 chunk	+0 lines, -6 lines	0 comments	Download
M	chrome/browser/component_updater/swiftshader_component_installer.cc	View	2 chunks	+0 lines, -5 lines	0 comments	Download
M	chrome/browser/ui/startup/obsolete_system_infobar_delegate.cc	View	3 chunks	+2 lines, -10 lines	0 comments	Download
M	chrome/browser/upgrade_detector_impl.cc	View	2 chunks	+0 lines, -6 lines	0 comments	Download
M	chrome/common/chrome_content_client.cc	View	3 chunks	+2 lines, -9 lines	0 comments	Download
M	skia/ext/convolver.h	View	1 chunk	+0 lines, -1 line	0 comments	Download
M	skia/ext/convolver.cc	View	1 chunk	+4 lines, -7 lines	0 comments	Download

Total messages: 5 (0 generated)

Lei Zhang

TBR since it's the same changes in chrome/ and skia/. I will boot up the ...

6 years, 3 months ago (2014-08-27 07:12:11 UTC) #1

commit-bot: I haz the power

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

6 years, 3 months ago (2014-08-27 07:12:29 UTC) #3

commit-bot: I haz the power

Committed patchset #1 (1) as 9d205596b821f6eeb12431975eb355741d8ad84a

6 years, 3 months ago (2014-08-27 08:36:54 UTC) #4

commit-bot: I haz the power

6 years, 3 months ago (2014-09-10 02:50:35 UTC) #5

Message was sent while issue was closed.

Patchset 1 (id:??) landed as
https://crrev.com/f2e5463e1d4f50bad11ee48056d6cb6037e20d1c
Cr-Commit-Position: refs/heads/master@{#292121}