[chromium-dev] Re: Mac ui_tests blows away from fraction of your profile.

2009-04-08 Thread Darin Fisher
Also, unless you do something like this, it ends up being difficult for
developers to dogfood the product, or if they do try to dogfood the product,
then they are strongly inscentivized not to run the ui_tests.  Neither
option is good.
-Darin


On Wed, Apr 8, 2009 at 11:21 AM, Darin Fisher da...@chromium.org wrote:

 Maybe this is well known, but what we did to avoid this problem on Windows
 is to leverage the --user-data-dir command line switch to force the chrome
 instance launched by the ui_tests to use a dedicated user data directory.
  We toss that directory prior to each test case IIRC.
 -Darin


 On Wed, Apr 8, 2009 at 11:10 AM, Scott Hess sh...@chromium.org wrote:


 I posted this on the irc channel yesterday, I know at least Peter
 noticed, but John suggested something more overt.

 When you run ui_tests on Mac, you will blow away some fraction of your
 Chromium.app profile.  Things like history and bookmarks.  This is a
 known issue, but may not be a known consequence.  If anyone wants to
 circle back and fix profiles, that would be wonderous.  I was thinking
 of looking at it myself, but am instead trying to figure out where my
 omnibox stuff is going wrong.

 It occurs to me that if you ran ui_tests while running Chromium.app,
 or two ui_tests, you may also see strange and wonderful results.

 -scott

 



--~--~-~--~~~---~--~~
Chromium Developers mailing list: chromium-dev@googlegroups.com 
View archives, change email options, or unsubscribe: 
http://groups.google.com/group/chromium-dev
-~--~~~~--~~--~--~---



[chromium-dev] Re: Mac ui_tests blows away from fraction of your profile.

2009-04-08 Thread Evan Martin

Note that a NOTIMPLEMENTED() prints when the code runs that tries to
abort when Chromium is already running.

One thing that I think has helped us on Linux is reducing the
NOTIMPLEMENTED prints to the bare minimum -- for stuff that we know is
broken (like the star button on the toolbar) having it print out a
warning on every page load masks important printouts like the one
above.  I have converted many such printouts into bugs instead (it's
obvious the star button doesn't* work if you click on it).

If you do the same, please be sure to tag it with the relevant OS
labels if it affects more than just Mac.  :)

* I think ours actually does work now; just an example.

On Wed, Apr 8, 2009 at 11:23 AM, Scott Hess sh...@chromium.org wrote:

 That would be my expectation, too.  Like I said, I think this is a
 known issue (the Mac port's profile stuff is sort of wired together).

 -scott


 On Wed, Apr 8, 2009 at 11:21 AM, Darin Fisher da...@chromium.org wrote:
 Maybe this is well known, but what we did to avoid this problem on Windows
 is to leverage the --user-data-dir command line switch to force the chrome
 instance launched by the ui_tests to use a dedicated user data directory.
  We toss that directory prior to each test case IIRC.
 -Darin

 On Wed, Apr 8, 2009 at 11:10 AM, Scott Hess sh...@chromium.org wrote:

 I posted this on the irc channel yesterday, I know at least Peter
 noticed, but John suggested something more overt.

 When you run ui_tests on Mac, you will blow away some fraction of your
 Chromium.app profile.  Things like history and bookmarks.  This is a
 known issue, but may not be a known consequence.  If anyone wants to
 circle back and fix profiles, that would be wonderous.  I was thinking
 of looking at it myself, but am instead trying to figure out where my
 omnibox stuff is going wrong.

 It occurs to me that if you ran ui_tests while running Chromium.app,
 or two ui_tests, you may also see strange and wonderful results.

 -scott

 



 


--~--~-~--~~~---~--~~
Chromium Developers mailing list: chromium-dev@googlegroups.com 
View archives, change email options, or unsubscribe: 
http://groups.google.com/group/chromium-dev
-~--~~~~--~~--~--~---



[chromium-dev] Re: Mac ui_tests blows away from fraction of your profile.

2009-04-08 Thread Scott Hess

I can see how that's happening in pref_service_uitest.cc and
memory_test.cc, but AFAICT I don't see how it's being done generally
for ui_tests test cases.  Perhaps it's that --user-data-dir should be
passed to ui_tests itself, which means if you run ui_tests without
that flag, it will still use your default profile.  Or maybe I
misunderstand entirely.

BTW, I was just stupid when I thought that Mac was broken because of
being hacked together.  Some of the profile setup is certainly hacked
together, but as far as I can tell, it does the right thing for
--user-data-dir (though having it fall back to the default dir when
the provided one doesn't exist threw me for awhile, that seems
unexpected).

-scott



On Wed, Apr 8, 2009 at 11:23 AM, Darin Fisher da...@chromium.org wrote:
 Also, unless you do something like this, it ends up being difficult for
 developers to dogfood the product, or if they do try to dogfood the product,
 then they are strongly inscentivized not to run the ui_tests.  Neither
 option is good.
 -Darin

 On Wed, Apr 8, 2009 at 11:21 AM, Darin Fisher da...@chromium.org wrote:

 Maybe this is well known, but what we did to avoid this problem on Windows
 is to leverage the --user-data-dir command line switch to force the chrome
 instance launched by the ui_tests to use a dedicated user data directory.
  We toss that directory prior to each test case IIRC.
 -Darin

 On Wed, Apr 8, 2009 at 11:10 AM, Scott Hess sh...@chromium.org wrote:

 I posted this on the irc channel yesterday, I know at least Peter
 noticed, but John suggested something more overt.

 When you run ui_tests on Mac, you will blow away some fraction of your
 Chromium.app profile.  Things like history and bookmarks.  This is a
 known issue, but may not be a known consequence.  If anyone wants to
 circle back and fix profiles, that would be wonderous.  I was thinking
 of looking at it myself, but am instead trying to figure out where my
 omnibox stuff is going wrong.

 It occurs to me that if you ran ui_tests while running Chromium.app,
 or two ui_tests, you may also see strange and wonderful results.

 -scott

 




--~--~-~--~~~---~--~~
Chromium Developers mailing list: chromium-dev@googlegroups.com 
View archives, change email options, or unsubscribe: 
http://groups.google.com/group/chromium-dev
-~--~~~~--~~--~--~---