Get rid of pixelSnapped* from IntRect. These are unused.

Source/core/platform/graphics/IntRect.cpp

 /*
  * Copyright (C) 2003, 2006, 2009 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 11 matching lines @@
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/platform/graphics/IntRect.h"
 
 #include <algorithm>
 #include "core/platform/graphics/FloatRect.h"
 #include "core/platform/graphics/LayoutRect.h"
+#include "wtf/Vector.h"

[pdr., 2013/06/07 17:17:14]

Is this needed?

[jbroman, 2013/06/07 17:22:28]

Since it's in the header, no. Didn't mean to add that here.

I have another CL which moves this line from the header to the .cpp file (but
this exposes transitive dependency issues that make that change too large for
this CL). Must have accidentally changed that line here too.

 
 using std::max;
 using std::min;
 
 namespace WebCore {
 
 IntRect::IntRect(const FloatRect& r)
     : m_location(clampToInteger(r.x()), clampToInteger(r.y()))
     , m_size(clampToInteger(r.width()), clampToInteger(r.height()))
 {
@@ skipping 111 matching lines @@
     IntRect result;
 
     size_t count = rects.size();
     for (size_t i = 0; i < count; ++i)
         result.unite(rects[i]);
 
     return result;
 }
 
 } // namespace WebCore