Add scale factor tag to data packs

ui/base/resource/data_pack.h

Index: ui/base/resource/data_pack.h
diff --git a/ui/base/resource/data_pack.h b/ui/base/resource/data_pack.h
index 5e3426f8b7df1dfdae813c8a6321b9d1d460bc68..50fe608cced47588b29a7d2b18ef5677ea3989d0 100644
--- a/ui/base/resource/data_pack.h
+++ b/ui/base/resource/data_pack.h
@@ -52,6 +52,9 @@ class UI_EXPORT DataPack : public ResourceHandle {
   virtual base::RefCountedStaticMemory* GetStaticMemory(
       uint16 resource_id) const OVERRIDE;
   virtual TextEncodingType GetTextEncodingType() const OVERRIDE;
+  virtual float GetScaleFactor() const OVERRIDE;
+
+  void set_scale_factor(float scale_factor) { scale_factor_ = scale_factor; }

[sky, 2012/04/25 21:11:42]

Do you really need a setter here? A setter implies you can change after the
fact, which I don't think we want, right? How about passing to constructor and
making const.

nit: non-virtual before virtual.

[sail, 2012/04/25 22:18:35]

Done.
Removed setter.

 
  private:
   // The memory-mapped data.
@@ -63,6 +66,9 @@ class UI_EXPORT DataPack : public ResourceHandle {
   // Type of encoding for text resources.
   TextEncodingType text_encoding_type_;
 
+  // The scale factor of image resources.

[sky, 2012/04/25 21:11:42]

This isn't really clear. It reads like all images are scaled by this, which I
don't think is what it does. 

[sail, 2012/04/25 22:18:35]

Done.

+  float scale_factor_;
+
   DISALLOW_COPY_AND_ASSIGN(DataPack);
 };