[FastTextAutosizer] Incorporate accessibility and device scale factors

Source/core/rendering/FastTextAutosizer.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 81 matching lines @@
         typedef WTF::HashMap<RenderBlock*, AtomicString> FingerprintMap;
         typedef WTF::HashMap<AtomicString, OwnPtr<BlockSet> > ReverseFingerprintMap;
 
         FingerprintMap m_fingerprints;
         ReverseFingerprintMap m_blocksForFingerprint;
     };
 
     explicit FastTextAutosizer(Document*);
 
     bool enabled();
-    void prepareWindowInfo(RenderView*);
+    void prepareRenderViewInfo(RenderView*);
     bool shouldBeClusterRoot(RenderBlock*);
     bool clusterWantsAutosizing(RenderBlock*);
     AtomicString computeFingerprint(RenderBlock*);
     Cluster* getOrCreateCluster(RenderBlock*);
     Cluster* createCluster(RenderBlock*);
     Cluster* addSupercluster(AtomicString, RenderBlock*);
     RenderBlock* deepestCommonAncestor(BlockSet&);
     float computeMultiplier(RenderBlock*);
     void applyMultiplier(RenderObject*, float);
 
     Document* m_document;
     int m_windowWidth; // Frame width in density-independent pixels (DIPs).
     int m_layoutWidth; // Layout width in CSS pixels.
+    float m_additionalMultiplier; // Includes accessibility font scale factor and device scale adjustment.

[skobes, 2014/01/13 19:10:13]

Perhaps "m_baseMultiplier"?

[pdr., 2014/01/13 19:49:12]

I like it. Done.

+#ifndef NDEBUG

[skobes, 2014/01/13 19:10:13]

Is the compiler not smart enough to strip out an unused field in the release
build?

[pdr., 2014/01/13 19:49:12]

Nowhere near smart enough :)

There are two reasons for using #ifndef NDEBUG here:
1) It will force a compile-time failure if we accidentally reference it outside
of a debug build
2) The compiler has a very local view of this class. For example, a separate
program could reference this object and would expect its layout (member
variables, etc) to be there. This is, of course, rare and maybe optimization
should be opt-out instead of opt-in, but that's a question for the c++ gods.

+    bool m_renderViewInfoPrepared; // Used for assertions.
+#endif
 
     // Clusters are created and destroyed during layout. The map key is the
     // cluster root. Clusters whose roots share the same fingerprint use the
     // same multiplier.
     ClusterMap m_clusters;
     ClusterStack m_clusterStack;
     FingerprintMapper m_fingerprintMapper;
 };
 
 } // namespace WebCore
 
 #endif // FastTextAutosizer_h