V8ParameterBase is redundant.
Created attachment 175159 [details] Patch
After landing this patch, I'll merge prepare() with V8Parameter's constructor (prepare() is always called just after constructing V8Parameter). I'll also rename V8Parameter to a better name (V8Parameter is just a "super" class for String and AtomicString).
Comment on attachment 175159 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=175159&action=review > Source/WebCore/bindings/v8/V8StringResource.h:52 > +template <V8ParameterMode MODE = DefaultMode> MODE -> Mode > Source/WebCore/bindings/v8/V8StringResource.h:55 > + V8Parameter(v8::Local<v8::Value> object) : m_v8Object(object), m_mode(Externalize), m_string() { } Please split this up into multiple lines.
Created attachment 175172 [details] patch for landing
Comment on attachment 175172 [details] patch for landing Clearing flags on attachment: 175172 Committed r135266: <http://trac.webkit.org/changeset/135266>
All reviewed patches have been landed. Closing bug.