Move nsimage_cache from base to app/mac. Use the app::mac namespace. Update c...

app/mac/nsimage_cache.h

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef BASE_NSIMAGE_CACHE_MAC_H_
-#define BASE_NSIMAGE_CACHE_MAC_H_
+#ifndef APP_MAC_NSIMAGE_CACHE_H_
+#define APP_MAC_NSIMAGE_CACHE_H_
 #pragma once
 
 #ifdef __OBJC__
 @class NSImage;
 @class NSString;
 #else
 class NSImage;
 class NSString;
 #endif
 
-namespace nsimage_cache {
+namespace app {
+namespace mac {
 
 // Returns an autoreleased image from the main app bundle
 // (mac_util::MainAppBundle()) with the given name, and keeps it in memory so
 // future fetches are fast.
 // NOTE:
 //   - This should only be called on the main thread.
 //   - The caller should retain the image if they want to keep it around, as
 //     the cache could have limit on size/lifetime, etc.
-NSImage* ImageNamed(NSString* name);
+NSImage* GetCachedImageWithName(NSString* name);

[Avi (use Gerrit), 2010/12/22 19:06:38]

Why the name change? This does imply a bit that only cached images are returned.

[brettw, 2010/12/22 19:12:05]

app::mac::ImageNamed is too vague (more stuff will move into here), and we're
trying to follow the style guide's path-based namespace scheme which implies no
one-off namespaces for random files like this used to have.

In some other places, we've used static functions in a class, which would make
this app::mac::ImageCache::ImageNamed but having longer function names seemed
more readable and just as sensible in this case.

 
-// Clears the cache.
-void Clear(void);
+// Clears the image cache.
+void ClearCachedImages(void);
 
-}  // namespace nsimage_cache
+}  // namespace mac
+}  // namespace app
 
-#endif  // BASE_NSIMAGE_CACHE_MAC_H_
+#endif  // APP_MAC_NSIMAGE_CACHE_H_