Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(411)

Issue 6902001: Update netflix version in chromeos plugin check (Closed)

Created:
9 years, 8 months ago by Nirnimesh
Modified:
9 years, 6 months ago
Reviewers:
krisr
CC:
chromium-reviews, Paweł Hajdan Jr.
Visibility:
Public.

Description

Update netflix version in chromeos plugin check R=krisr@chromium.org BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=82902

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/test/data/chromeos_plugins_list.txt View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Nirnimesh
9 years, 8 months ago (2011-04-25 18:17:44 UTC) #1
krisr
9 years, 8 months ago (2011-04-25 18:31:52 UTC) #2
LGTM

On Mon, Apr 25, 2011 at 11:17 AM, <nirnimesh@chromium.org> wrote:

> Reviewers: krisr,
>
> Description:
> Update netflix version in chromeos plugin check
>
>
> R=krisr@chromium.org
> BUG=
> TEST=
>
>
> Please review this at http://codereview.chromium.org/6902001/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files:
>  M chrome/test/data/chromeos_plugins_list.txt
>
>
> Index: chrome/test/data/chromeos_plugins_list.txt
> diff --git a/chrome/test/data/chromeos_plugins_list.txt
> b/chrome/test/data/chromeos_plugins_list.txt
> index
>
df6bf92df9bf95094c3ad996cc293dd1c5b3ee25..df1938026fab01026ea84ce00a4a54350fa2bb6c
> 100644
> --- a/chrome/test/data/chromeos_plugins_list.txt
> +++ b/chrome/test/data/chromeos_plugins_list.txt
> @@ -8,7 +8,7 @@
>  {u'unique_key':'path', u'name':u'Shockwave Flash',
> u'path':'/opt/google/chrome/pepper/libpepflashplayer.so', u'enabled':1,
> u'boards':['x86-mario', 'x86-alex']},
>  {u'unique_key':'path', u'name':u'Shockwave Flash',
> u'path':'/opt/google/chrome/plugins/libgcflashplayer.so', u'enabled':1,
> u'boards':['x86-mario', 'x86-alex']},
>  {u'name':'Chrome PDF Viewer', u'enabled':1, u'version':u'',
> u'boards':['x86-mario', 'x86-alex']},
> - {u'name':'Netflix', u'enabled':1, u'version':u'0.1.1',
> u'boards':['x86-alex']},
> + {u'name':'Netflix', u'enabled':1, u'version':u'0.2.0',
> u'boards':['x86-alex']},
>  {u'name':'Google Talk Plugin', u'enabled':1, u'version':u'',
> u'boards':['x86-mario', 'x86-alex']},
>  {u'name':'O3D Plugin', u'enabled':1, u'version':u'',
> u'boards':['x86-mario', 'x86-alex']},
>  {u'name':'Default Plug-in', u'enabled':1, u'version':u'1',
> u'boards':['x86-mario', 'x86-alex']},
>
>
>

Powered by Google App Engine
This is Rietveld 408576698