Bug 87798

Summary:

[chromium] Merge shared code in CCOcclusionTrackerTest

Product:

WebKit

Reporter:

Shawn Singh <shawnsingh>

Component:

Layout and Rendering

Assignee:

Shawn Singh <shawnsingh>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

danakj, enne, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Shawn Singh 2012-05-29 17:32:28 PDT

Patch coming in a moment.  It might look like a large patch, but its actually a tiny minor trivial change... took only 5 minutes to create the patch, and 5 minutes to test it.

Are you guys OK with doing this?  Personally I think its the right thing to do.   But please feel free to WONTFIX it if you guys don't want this change for some reason.

Comment 1 Shawn Singh 2012-05-29 17:34:06 PDT

Created attachment 144658 [details]
Patch

Comment 2 Dana Jansens 2012-05-29 18:09:34 PDT

woot, i'm fine with this.

Comment 3 Adrienne Walker 2012-05-30 13:07:16 PDT

Comment on attachment 144658 [details]
Patch

Sure! R=me.

Comment 4 WebKit Review Bot 2012-05-30 13:17:27 PDT

Comment on attachment 144658 [details]
Patch

Clearing flags on attachment: 144658

Committed r118959: <http://trac.webkit.org/changeset/118959>

Comment 5 WebKit Review Bot 2012-05-30 13:17:31 PDT

All reviewed patches have been landed.  Closing bug.