For example, OptionsObject::getKeyBool() is an alias of OptionsObject::getKeyValue(const String& key, bool& value). We should remove OptionsObject::getKeyXXXX() (XXXX is some specific type) and unify them into OptionsObject::getKeyValue(const String& key, XXXX& value). c.f. Corresponding JSC methods are unified into JSDictionary::convertValue(JSC::ExecState*, JSC::JSValue, XXXX&).
Created attachment 111890 [details] Patch
Comment on attachment 111890 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=111890&action=review > Source/WebCore/bindings/v8/OptionsObject.h:54 > + bool getKeyValue(const String&, bool&) const; > + bool getKeyValue(const String&, int32_t&) const; > + bool getKeyValue(const String&, double&) const; > + bool getKeyValue(const String&, String&) const; > + bool getKeyValue(const String&, ScriptValue&) const; Why not just call these functions "get" ? > Source/WebCore/bindings/v8/OptionsObject.h:61 > + bool getKeyValueWithUndefinedOrNullCheck(const String&, String&) const; getWithUndefinedOrNullCheck ?
Created attachment 112624 [details] patch for commit
(In reply to comment #2) > (From update of attachment 111890 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=111890&action=review > > > Source/WebCore/bindings/v8/OptionsObject.h:54 > > + bool getKeyValue(const String&, bool&) const; > > + bool getKeyValue(const String&, int32_t&) const; > > + bool getKeyValue(const String&, double&) const; > > + bool getKeyValue(const String&, String&) const; > > + bool getKeyValue(const String&, ScriptValue&) const; > > Why not just call these functions "get" ? > > > Source/WebCore/bindings/v8/OptionsObject.h:61 > > + bool getKeyValueWithUndefinedOrNullCheck(const String&, String&) const; > > getWithUndefinedOrNullCheck ? Done and committed. Thanks!!
Comment on attachment 112624 [details] patch for commit Clearing flags on attachment: 112624 Committed r98572: <http://trac.webkit.org/changeset/98572>
Attachment 111890 [details] was posted by a committer and has review+, assigning to Kentaro Hara for commit.