[webkit-reviews] review granted: [Bug 77033] Implement overridePreference for boolean preferences in WebKitTestRunner : [Attachment 124020] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 25 15:18:54 PST 2012


Alexey Proskuryakov <ap at webkit.org> has granted Caio Marcelo de Oliveira Filho
<cmarcelo at webkit.org>'s request for review:
Bug 77033: Implement overridePreference for boolean preferences in
WebKitTestRunner
https://bugs.webkit.org/show_bug.cgi?id=77033

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

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
> I made a small change to the initial patch: instead of overwriting the
preferences, the getter now asks the override first.

Thanks! I'd appreciate a detailed explanation for posterity - I think that I
understand what you're saying, but only barely, and it will be more difficult
in a few months. I.e., what function gets called between tests, what
preferences it resets, and how that filters down to each page.


More information about the webkit-reviews mailing list