[webkit-reviews] review granted: [Bug 28830] [Gtk] DRT needs implementation of overridePreference : [Attachment 38780] Patch v1

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has granted Jan Alonzo <jmalonzo at gmail.com>'s
request for review:
Bug 28830: [Gtk] DRT needs implementation of overridePreference
https://bugs.webkit.org/show_bug.cgi?id=28830

Attachment 38780: Patch v1
https://bugs.webkit.org/attachment.cgi?id=38780&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
Looks great!

I would have named "toWebSettingKey" "copyAsWebSettingKey" or something.  To
inform the caller that they need to release the returned string.

It's so sad that we don't have some sort of GOwnPtr for DRT to use here instead
of needing to manually g_free.

Spacing:
+	     if ( !(ss >> val).fail())

+    static gboolean isInitialized = false;
is not really needed, you can check !keyTable.

Please adjust a little before landing.