[Webkit-unassigned] [Bug 13467] REGRESSION (r20972): Google Spreadsheets: spreadsheet controls not displayed

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=13467





------- Comment #2 from koivisto at iki.fi  2007-05-02 06:16 PDT -------
This was broken by r20972 which removed document property from iframe element.
This was done to fix a bug with SAP application. The new behavior should match
Firefox.

The problem is that after this change the spreadsheet grid gets loaded to main
view instead of the iframe it should go to.

Looking around the obfuscated code in
http://spreadsheets.google.com/client/js/2100283068-trix_main.js I saw the
following function:

  ff[_P].hj = function ()
  {
    if ($b)
      {
        if (this.Ui[hb] == null)
          {
            return null;
          }
        return this.Ui[hb][lg];
      }
    else
      {
        return this.Ui[jg];
      }
  };

Here $b is a Safari test result, hb="document", lg="defaultView" and
jg="contentWindow" and this.Ui is an iframe element (I think). Basically for
Safari it does "iframe.document.defaultView" whereas for all other browsers it
does "iframe.contentWindow". With iframe.document gone, this would not work
anymore. I can't verify that this exact function causes the problem (there are
other Safari tests and other places where iframe content is accessed in various
ways) but it certainly looks like something which would fail after the change
in r20972.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.