There would still be an option to run in all frames, it just wouldn't
be on by default.

- a

On Thu, Nov 19, 2009 at 2:58 PM, Robert Billingslea
<robert.billings...@gmail.com> wrote:
> Correct me if I am wrong, but if content scripts were only run in the top
> frame, I believe we won't be able to access sub-frames that were loaded from
> different domains (because of javascript domain origin restrictions).
> That would be bad for us.
>
> On Thu, Nov 19, 2009 at 4:19 PM, Michael Weber
> <michaelhermannwe...@gmail.com> wrote:
>>
>> I think either way is fine. It is easy enough to filter frames using
>> "if (window == top)".
>>
>> A note in the docs to emphasize the behaviour would help.
>>
>> On Nov 19, 1:11 pm, Aaron Boodman <a...@google.com> wrote:
>> > On Thu, Nov 19, 2009 at 12:42 PM, Michael Weber
>> >
>> > <michaelhermannwe...@gmail.com> wrote:
>> > > I don't see any mention of it in the docs, but it makes sense and
>> > > seems to be what we are seeing.
>> >
>> > > I was seeing some unexpected behaviour that is fixed by including a an
>> > > "if (window == top)" in our content script to ensure it only executes
>> > > for the root frame.
>> >
>> > > In this case, the follow issues I logged are complete bogus and I will
>> > > update them, and then run back to my corner with my tail between my
>> > > legs.
>> >
>> > > 28148: document HTML sometimes not set correctly in DOM
>> > > 26513: document url not set correctly in DOM
>> >
>> > Yes, content scripts are expected to be run in all frames. I wonder if
>> > we should change this and make it default to root only. It seems to
>> > confuse a lot of people.
>> >
>> > - a
>>
>> --
>>
>> You received this message because you are subscribed to the Google Groups
>> "Chromium-extensions" group.
>> To post to this group, send email to chromium-extensi...@googlegroups.com.
>> To unsubscribe from this group, send email to
>> chromium-extensions+unsubscr...@googlegroups.com.
>> For more options, visit this group at
>> http://groups.google.com/group/chromium-extensions?hl=.
>>
>>
>
> --
>
> You received this message because you are subscribed to the Google Groups
> "Chromium-extensions" group.
> To post to this group, send email to chromium-extensi...@googlegroups.com.
> To unsubscribe from this group, send email to
> chromium-extensions+unsubscr...@googlegroups.com.
> For more options, visit this group at
> http://groups.google.com/group/chromium-extensions?hl=.
>

--

You received this message because you are subscribed to the Google Groups 
"Chromium-extensions" group.
To post to this group, send email to chromium-extensi...@googlegroups.com.
To unsubscribe from this group, send email to 
chromium-extensions+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/chromium-extensions?hl=.


Reply via email to