Remove references to IsNewAvatarMenu since the flag was removed.

chrome/browser/ui/cocoa/cocoa_profile_test.h

Index: chrome/browser/ui/cocoa/cocoa_profile_test.h
diff --git a/chrome/browser/ui/cocoa/cocoa_profile_test.h b/chrome/browser/ui/cocoa/cocoa_profile_test.h
index 99a46640102fc91f5ffdf24056d3814e5711982e..9c3b35a936dcd575fc5f269d4bf461c5be32e5d0 100644
--- a/chrome/browser/ui/cocoa/cocoa_profile_test.h
+++ b/chrome/browser/ui/cocoa/cocoa_profile_test.h
@@ -33,6 +33,7 @@ class TestingProfile;
 class CocoaProfileTest : public CocoaTest {
  public:
   CocoaProfileTest();
+  CocoaProfileTest(bool incognito);

[Robert Sesek, 2015/09/28 15:14:47]

Is this change to CocoaProfileTest really necessary? Why can't you just create a
new one-off browser in the one test where you use this ctor?

[anthonyvd, 2015/09/29 20:23:15]

I would end up pretty much copy-pasting the CocoaProfileTest and the CocoaTest
browser creation logic so I figured that it might as well be added here since it
reduces duplication and allows other eventual tests to benefit from it. If you
feel like it doesn't belong I'll be happy to move it to be specific to my test
:)

[Robert Sesek, 2015/09/29 20:31:11]

You can still inherit from CocoaProfileTest, but you just need to create a new
browser using the fixture's profile(). I don't think this belongs in a common
superclass if there's only one user.

[anthonyvd, 2015/09/30 18:34:03]

Fair point, done. I don't have access to my mac so I'll run some trybots to see
if I did it correctly.

   ~CocoaProfileTest() override;
 
   // This constructs a a Browser and a TestingProfile. It is guaranteed to
@@ -72,6 +73,7 @@ class CocoaProfileTest : public CocoaTest {
   TestingProfile* profile_;  // Weak; owned by profile_manager_.
   TestingProfile::TestingFactories testing_factories_;
   scoped_ptr<Browser> browser_;
+  bool incognito_;
 
   scoped_ptr<content::TestBrowserThreadBundle> thread_bundle_;
 };