You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to chromi...@chromium.org
Hi,
I'm making a change that adds functionality to net/ that's not
expected to work under Chrome Frame (it relies on code in Chromium's
URL request jobs, which aren't run in Chrome Frame). I've taken a
look around chrome_frame.gyp and other tests, but can't find a way to
tell in my test cases whether I'm running in a Chrome Frame context or
elsewhere. Does anyone have any ideas?
Thanks,
ttuttle
Greg Thompson
unread,
May 17, 2013, 3:41:27 PM5/17/13
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Greg Thompson, chromium-dev
I don't want to remove them entirely, since some are modifications of
existing tests. I might fork the old ones and disable only the new
ones.
Matthew Menke
unread,
May 17, 2013, 11:39:20 PM5/17/13
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to chromi...@chromium.org, Greg Thompson
Having code in net/ test for the presence of Chrome Frame seems like it would be a layering violation.
Also, you can disable only specific tests in ChromeFrame, so you can, in fact, just disable the new ones - see src/chrome_frame/test/net/fake_external_tab.cc