Issue 15713012: Initialize CPU-specific features in media when starting the host.

Issue 15713012: Initialize CPU-specific features in media when starting the host. (Closed)

Created:
7 years, 6 months ago by alexeypa (please no reviews)

Modified:
7 years, 6 months ago

Reviewers:
Wez

CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, rmsousa+watch_chromium.org, weitaosu+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org

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

Visibility:
Public.

More Reviews

Description

Initialize CPU-specific features in media when starting the host. BUG=245324 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203214

Patch Set 1 #

Total comments: 2

Created: 7 years, 6 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+4 lines, -0 lines)			Patch
	M	remoting/host/remoting_me2me_host.cc	View		2 chunks	+4 lines, -0 lines	2 comments	Download

Messages

Total messages: 5 (0 generated)

Expand Messages | Collapse Messages

Wez

lgtm https://codereview.chromium.org/15713012/diff/1/remoting/host/remoting_me2me_host.cc File remoting/host/remoting_me2me_host.cc (right): https://codereview.chromium.org/15713012/diff/1/remoting/host/remoting_me2me_host.cc#newcode1050 remoting/host/remoting_me2me_host.cc:1050: media::InitializeCPUSpecificMediaFeatures(); This looks reasonable given the bug description, ...

7 years, 6 months ago (2013-05-30 17:17:29 UTC) #2

alexeypa (please no reviews)

https://codereview.chromium.org/15713012/diff/1/remoting/host/remoting_me2me_host.cc File remoting/host/remoting_me2me_host.cc (right): https://codereview.chromium.org/15713012/diff/1/remoting/host/remoting_me2me_host.cc#newcode1050 remoting/host/remoting_me2me_host.cc:1050: media::InitializeCPUSpecificMediaFeatures(); On 2013/05/30 17:17:29, Wez wrote: > This looks ...

7 years, 6 months ago (2013-05-30 17:35:01 UTC) #3

commit-bot: I haz the power

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

7 years, 6 months ago (2013-05-30 17:35:19 UTC) #4

Message was sent while issue was closed.

Change committed as 203214

Expand Messages | Collapse Messages