[Webkit-unassigned] [Bug 41529] Add the ability for user scripts and user styles to affect just the top frame

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 8 10:36:24 PDT 2010


Timothy Hatcher <timothy at apple.com> changed:

           What    |Removed                     |Added
  Attachment #60786|review?                     |review+, commit-queue-
               Flag|                            |

--- Comment #14 from Timothy Hatcher <timothy at apple.com>  2010-07-08 10:36:24 PST ---
(From update of attachment 60786)
 +              if (sheet->injectedFrames() == InjectInTopFrameOnly && ownerElement())
You should add a test using <frameset>. I'm curious if ownerElement() does the right thing there. Arguably all the frames in a frameset are top level.

 +                     UserScriptInjectedFrames injectedFrames)
It is odd to use UserScriptInjectedFrames with stylesheets. Maybe UserContentInjectedFrames? Is there a shared header it could move to as well?

 +                     injectedFrames:(WebUserScriptInjectedFrames)injectedFrames
It is odd to use WebUserScriptInjectedFrames for the stylesheet case too. Maybe WebUserContentInjectedFrames?

 +  // FIXME: The following two methods are deprecated in favor of the overloads below that take the WebUserScriptInjectedFrames argument. https://bugs.webkit.org/show_bug.cgi?id=41800.
I don't think the bug is needed.

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

More information about the webkit-unassigned mailing list