[Webkit-unassigned] [Bug 67584] [Chromium] Add WebSandboxSupport and WebThemeEngine for Android.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=67584





--- Comment #12 from Steve Block <steveblock at google.com>  2011-09-06 02:40:20 PST ---
(From update of attachment 106385)
View in context: https://bugs.webkit.org/attachment.cgi?id=106385&action=review

LGTM

> Source/WebKit/chromium/public/android/WebSandboxSupport.h:2
> + * Copyright (C) 2010 Google Inc. All rights reserved.

s/2010/2011

> Source/WebKit/chromium/public/android/WebSandboxSupport.h:36
> +// FIXME: Empty class for now, as we need it to compile.

Please link to a bug so this ugliness doesn't get forgotten about.

> Source/WebKit/chromium/public/android/WebSandboxSupport.h:37
> +// Put methods here that are required due to sandbox restrictions.

I don't think this comment is needed

> Source/WebKit/chromium/public/android/WebThemeEngine.h:2
> + * Copyright (C) 2010 Google Inc. All rights reserved.

s/2010/2011

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.