rolling tree back to its state as of r2169 to see if that fixes Windows buildbots

git-svn-id: http://skia.googlecode.com/svn/trunk@2179 2bbb7eff-a529-9590-31e7-b0007b416f81

gm/points.cpp

@@ -28,14 +28,8 @@ protected:
     }
 
     static void fill_pts(SkPoint pts[], size_t n, SkRandom* rand) {
-        for (size_t i = 0; i < n; i++) {
-            // Compute these independently and store in variables, rather
-            // than in the parameter-passing expression, to get consistent
-            // evaluation order across compilers.
-            float y = rand->nextUScalar1() * 480;
-            float x = rand->nextUScalar1() * 640;
-            pts[i].set(x, y);
-        }
+        for (size_t i = 0; i < n; i++)
+            pts[i].set(rand->nextUScalar1() * 640, rand->nextUScalar1() * 480);
     }
 
     virtual void onDraw(SkCanvas* canvas) {

gm/strokerects.cpp

@@ -38,11 +38,9 @@ protected:
         SkScalar y = rand.nextUScalar1() * H;
         SkScalar w = rand.nextUScalar1() * (W >> 2);
         SkScalar h = rand.nextUScalar1() * (H >> 2);
-        SkScalar hoffset = rand.nextSScalar1();
-        SkScalar woffset = rand.nextSScalar1();
         
         r->set(x, y, x + w, y + h);
-        r->offset(-w/2 + woffset, -h/2 + hoffset);
+        r->offset(-w/2 + rand.nextSScalar1(), -h/2 +  + rand.nextSScalar1());
     }
 
     virtual void onDraw(SkCanvas* canvas) {

gm/strokes.cpp

@@ -25,11 +25,9 @@ static void rnd_rect(SkRect* r, SkPaint* paint, SkRandom& rand) {
     SkScalar y = rand.nextUScalar1() * H;
     SkScalar w = rand.nextUScalar1() * (W >> 2);
     SkScalar h = rand.nextUScalar1() * (H >> 2);
-    SkScalar hoffset = rand.nextSScalar1();
-    SkScalar woffset = rand.nextSScalar1();
     
     r->set(x, y, x + w, y + h);
-    r->offset(-w/2 + woffset, -h/2 + hoffset);
+    r->offset(-w/2 + rand.nextSScalar1(), -h/2 +  + rand.nextSScalar1());
     
     paint->setColor(rand.nextU());
     paint->setAlpha(0xFF);

gpu/src/GrBinHashKey.h

@@ -36,7 +36,7 @@ public:
         return *this;
     }
 
-    ~GrBinHashKey() {
+    virtual ~GrBinHashKey() {
     }
 
     void setKeyData(const uint32_t *data) {

gpu/src/GrClip.cpp

@@ -138,9 +138,13 @@ void GrClip::setFromIterator(GrClipIterator* iter, GrScalar tx, GrScalar ty,
         }
     }
     fConservativeBoundsValid = false;
-    if (isectRectValid && rectCount) {
-        fConservativeBounds = fList[0].fRect;
+    if (isectRectValid) {
         fConservativeBoundsValid = true;
+        if (rectCount > 0) {
+            fConservativeBounds = fList[0].fRect;
+        } else {
+            fConservativeBounds.setEmpty();
+        }
     } else if (NULL != conservativeBounds) {
         fConservativeBounds = *conservativeBounds;
         fConservativeBoundsValid = true;

gpu/src/GrGpuFactory.cpp

@@ -22,7 +22,7 @@
 
 GrGpu* GrGpu::Create(GrEngine engine, GrPlatform3DContext context3D) {
 
-    const GrGLInterface* glInterface = NULL;
+    const GrGLInterface* glInterface;
 
     if (kOpenGL_Shaders_GrEngine == engine ||
         kOpenGL_Fixed_GrEngine == engine) {

include/pdf/SkPDFDevice.h

@@ -118,7 +118,7 @@ public:
      *  clipped. A simple way to avoid the bug is to always draw the margin
      *  content last.
      */
-    SK_API void setDrawingArea(DrawingArea drawingArea);
+    void setDrawingArea(DrawingArea drawingArea);
 
     // PDF specific methods.
 

src/core/SkCanvas.cpp

@@ -528,6 +528,7 @@ SkDevice* SkCanvas::setDevice(SkDevice* device) {
         while ((rec = (MCRec*)iter.next()) != NULL) {
             (void)rec->fRegion->op(bounds, SkRegion::kIntersect_Op);
         }
+        fClipStack.clipDevRect(bounds, SkRegion::kIntersect_Op);
     }
     return device;
 }
@@ -1383,11 +1384,11 @@ private:
 
 void SkCanvas::DrawRect(const SkDraw& draw, const SkPaint& paint,
                         const SkRect& r, SkScalar textSize) {
-    if (paint.getStyle() == SkPaint::kFill_Style) {
+    if (paint.getStyle() == SkPaint::kFill_Style) {
         draw.fDevice->drawRect(draw, r, paint);
     } else {
         SkPaint p(paint);
-        p.setStrokeWidth(SkScalarMul(textSize, paint.getStrokeWidth()));
+        p.setStrokeWidth(SkScalarMul(textSize, paint.getStrokeWidth()));
         draw.fDevice->drawRect(draw, r, p);
     }
 }

src/pdf/SkPDFDevice.cpp

@@ -517,6 +517,7 @@ SkPDFDevice::SkPDFDevice(const SkISize& pageSize, const SkISize& contentSize,
     fInitialTransform.preConcat(initialTransform);
 
     SkIRect existingClip = SkIRect::MakeWH(this->width(), this->height());
+    fExistingClipStack.clipDevRect(existingClip);
     fExistingClipRegion.setRect(existingClip);
 
     this->init();

src/pdf/SkPDFFont.cpp

@@ -488,7 +488,7 @@ void append_cmap_sections(const SkTDArray<SkUnichar>& glyphToUnicode,
                             (i >> 8) == (currentRangeEntry.fStart >> 8) &&
                             glyphToUnicode[i] == (currentRangeEntry.fUnicode +
                                                   continuousEntries)) {
-                    currentRangeEntry.fEnd = i;
+                            currentRangeEntry.fEnd = i;
                     if (i == glyphToUnicode.count() - 1) {
                         // Last entry is in a range.
                         bfrangeEntries.push(currentRangeEntry);

src/ports/SkFontHost_linux.cpp

@@ -129,7 +129,7 @@ static SkTypeface* find_from_uniqueID(uint32_t uniqueID) {
         }
         curr = curr->fNext;
     }
-    return NULL;
+    return false;
 }
 
 static bool valid_uniqueID(uint32_t uniqueID) {

src/utils/SkParse.cpp

@@ -130,7 +130,7 @@ const char* SkParse::FindHex(const char str[], uint32_t* value)
             *value = n;
         return str;
     }
-    return NULL;
+    return false;
 }
 
 const char* SkParse::FindS32(const char str[], int32_t* value)

src/views/SkView.cpp

@@ -363,7 +363,7 @@ void SkView::Click::copyType(const char type[])
 SkView::Click* SkView::findClickHandler(SkScalar x, SkScalar y)
 {
 	if (x < 0 || y < 0 || x >= fWidth || y >= fHeight) {
-		return NULL;
+		return false;
     }
 
     if (this->onSendClickToChildren(x, y)) {

whitespace.txt

@@ -16,7 +16,4 @@ You can modify this file to create no-op changelists.
 
 
 
-
-
-