Oilpan: Remove unnecessary trace methods in the supplement hierarchy.

Oilpan: Remove unnecessary trace methods in the supplement hierarchy.

R=tkent@chromium.org
BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=173617

[haraken, 2014-05-07 12:22:44]

LGTM.

https://codereview.chromium.org/270213006/diff/1/Source/modules/screen_orient...
File Source/modules/screen_orientation/ScreenOrientationController.h (left):

https://codereview.chromium.org/270213006/diff/1/Source/modules/screen_orient...
Source/modules/screen_orientation/ScreenOrientationController.h:28: virtual void
trace(Visitor* visitor) OVERRIDE { DocumentSupplement::trace(visitor); }

Nit: We'll need this once we make Document& a Member.

[zerny-chromium, 2014-05-07 12:45:29]

Thanks for the review.

On 2014/05/07 12:22:44, haraken wrote:
> LGTM.
> 
>
https://codereview.chromium.org/270213006/diff/1/Source/modules/screen_orient...
> File Source/modules/screen_orientation/ScreenOrientationController.h (left):
> 
>
https://codereview.chromium.org/270213006/diff/1/Source/modules/screen_orient...
> Source/modules/screen_orientation/ScreenOrientationController.h:28: virtual
void
> trace(Visitor* visitor) OVERRIDE { DocumentSupplement::trace(visitor); }
> 
> Nit: We'll need this once we make Document& a Member.

I'll let that future CL re-add it when needed.

[tkent, 2014-05-07 13:34:09]

lgtm

[zerny-chromium, 2014-05-08 06:09:42]

The CQ bit was checked by zerny@chromium.org

[commit-bot: I haz the power, 2014-05-08 06:10:19]

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

[commit-bot: I haz the power, 2014-05-08 06:58:27]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  linux_blink_dbg on tryserver.blink

[commit-bot: I haz the power, 2014-05-08 08:20:58]

Change committed as 173617