Update resource_bundle.h image function comment.

ui/base/resource/resource_bundle.h

 // Copyright (c) 2012 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 UI_BASE_RESOURCE_RESOURCE_BUNDLE_H_
 #define UI_BASE_RESOURCE_RESOURCE_BUNDLE_H_
 
 #include "build/build_config.h"
 
 #include <map>
@@ skipping 159 matching lines @@
   // bitmap is owned by the resource bundle and should not be freed.
   //
   // !! THIS IS DEPRECATED. PLEASE USE THE METHOD BELOW. !!
   SkBitmap* GetBitmapNamed(int resource_id);
 
   // Gets image with the specified resource_id from the current module data.
   // Returns a pointer to a shared instance of gfx::ImageSkia. This shared
   // instance is owned by the resource bundle and should not be freed.
   // TODO(pkotwicz): Make method return const gfx::ImageSkia*
   //
-  // NOTE: It is preferrable to use GetImageNamed such that code is more
-  // portable.
+  // NOTE: GetImageNamed is preferred for cross-platform gfx::Image use,

[Robert Sesek, 2012/09/20 17:37:35]

GetNativeImageNamed, per below.

[sail, 2012/09/20 17:40:04]

I don't think this comment is correct.

Unless you're doing WebUI or there's some exceptional reason, you should be
using gfx::Image. This means Views code should be using gfx::Image.

At least, that's what I got out of this comment:

+  // but GetImageSkiaNamed is preferred for Views-specific use.
   gfx::ImageSkia* GetImageSkiaNamed(int resource_id);
 
   // Gets an image resource from the current module data. This will load the
   // image in Skia format by default. The ResourceBundle owns this.
   gfx::Image& GetImageNamed(int resource_id);
 
   // Similar to GetImageNamed, but rather than loading the image in Skia format,
   // it will load in the native platform type. This can avoid conversion from
   // one image type to another. ResourceBundle owns the result.
   //
@@ skipping 141 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(ResourceBundle);
 };
 
 }  // namespace ui
 
 // TODO(beng): Someday, maybe, get rid of this.
 using ui::ResourceBundle;
 
 #endif  // UI_BASE_RESOURCE_RESOURCE_BUNDLE_H_