Remove useless SkASSERT in SkRect.h and SkRegion.h

Remove useless SkASSERT in SkRect.h and SkRegion.h

The pointer of reference is always not-null. It causes >50k lines of warnning when chromium is compiled by clang.
Warning looks like
../../third_party/skia/include/core/SkRect.h:284:19: warning: reference cannot be bound to dereferenced null pointer in well-defined C++ code; pointer may be assumed to always convert to true [-Wundefined-bool-conversion]
    do { if (&r) break; SkNO_RETURN_HINT(); SkDebugf_FileLine("../../third_party/skia/include/core/SkRect.h", 284, false, "%s:%d: failed assertion \"%s\"\n", "../../third_party/skia/include/core/SkRect.h", 284, "&r"); SkDebugf_FileLine("../../third_party/skia/include/core/SkRect.h", 284, true, "SK_CRASH"); } while (false);

Committed: https://skia.googlesource.com/skia/+/79612de00f66408cff253605fbe2bb2a9e2b27ff

[dshwang, 2015-04-28 17:50:54]

dongseong.hwang@intel.com changed reviewers:
+ bsalomon@google.com, reed@google.com

[dshwang, 2015-04-28 17:50:54]

reed, bsalomon: could you review this simple CL?

[bsalomon, 2015-04-28 19:27:58]

lgtm

[reed1, 2015-04-28 19:40:03]

The CQ bit was checked by reed@google.com

[reed1, 2015-04-28 19:40:03]

lgtm

[commit-bot: I haz the power, 2015-04-28 19:40:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1111013002/1

[commit-bot: I haz the power, 2015-04-28 19:47:53]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/79612de00f66408cff253605fbe2bb2a9e2b27ff