Chromium Code Reviews

Issue 2931002: Pull PPAPI to get the paint aggregator, hook up UI test. (Closed)

Created:
10 years, 5 months ago by brettw
Modified:
9 years, 7 months ago
Reviewers:
jam
CC:
chromium-reviews, Paweł Hajdan Jr.
Visibility:
Public.

Description

Pull PPAPI to get the paint aggregator, hook up UI test. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=51902

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Stats (+5 lines, -1 line)
M DEPS View 1 chunk +1 line, -1 line 0 comments
M chrome/test/ui/ppapi_uitest.cc View 1 chunk +4 lines, -0 lines 0 comments

Messages

Total messages: 2 (0 generated)
brettw
10 years, 5 months ago (2010-07-08 17:54:55 UTC) #1
jam
10 years, 5 months ago (2010-07-08 17:58:17 UTC) #2
lgtm

On Thu, Jul 8, 2010 at 10:54 AM, <brettw@chromium.org> wrote:

> Reviewers: John Abd-El-Malek,
>
> Description:
> Pull PPAPI to get the paint aggregator, hook up UI test.
>
> Please review this at http://codereview.chromium.org/2931002/show
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     DEPS
>  M     chrome/test/ui/ppapi_uitest.cc
>
>
> Index: DEPS
> ===================================================================
> --- DEPS        (revision 51862)
> +++ DEPS        (working copy)
> @@ -158,7 +158,7 @@
>     Var("libvpx_revision"),
>
>   "src/third_party/ppapi":
> -    "http://ppapi.googlecode.com/svn/trunk@130",
> +    "http://ppapi.googlecode.com/svn/trunk@133",
>
>   "src/third_party/libjingle/source":
>     "http://libjingle.googlecode.com/svn/branches/nextsnap@" +
> Index: chrome/test/ui/ppapi_uitest.cc
> ===================================================================
> --- chrome/test/ui/ppapi_uitest.cc      (revision 51862)
> +++ chrome/test/ui/ppapi_uitest.cc      (working copy)
> @@ -118,6 +118,10 @@
>   RunTestViaHTTP("URLLoader");
>  }
>
> +TEST_F(PPAPITest, PaintAgggregator) {
> +  RunTestViaHTTP("PaintAggregator");
> +}
> +
>  // http://crbug.com/48544
>  TEST_F(PPAPITest, FAILS_Scrollbar) {
>   RunTest("Scrollbar");
>
>
>

Powered by Google App Engine