[webkit-reviews] review requested: [Bug 9673] Add support for window.atob() and window.btoa() : [Attachment 11405] proposed fix

[bugzilla-request-daemon at macosforge.org]

Alexey Proskuryakov <ap at webkit.org> has asked  for review:
Bug 9673: Add support for window.atob() and window.btoa()
http://bugs.webkit.org/show_bug.cgi?id=9673

Attachment 11405: proposed fix
http://bugs.webkit.org/attachment.cgi?id=11405&action=edit

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
This implementation is based on the one from KDE, but has been tweaked for
better Firefox compatibility.

One thing I couldn't understand in khtml (and thus didn't copy) was a special
case for AToB/BToA in Window::getOwnPropertySlot:

    switch(entry->value) {
    case Frames:
// ...
    case Blur:
    case AToB:
    case BToA:
      getSlotFromEntry<WindowFunc, Window>(entry, this, slot);

<http://websvn.kde.org/trunk/KDE/kdelibs/khtml/ecma/kjs_window.cpp?rev=592458&v
iew=markup>

WebCore has similar (but not identical) code for Blur etc. here, so maybe we
actually need it, too.