cc: Refactor in picture_pile.cc

cc: Refactor in picture_pile.cc

This is a minor refactor, that changes function names.

BUG=NONE

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282244

[hyunki, 2014-07-09 17:36:38]

PTAL. Thanks. :)

[enne (OOO), 2014-07-09 17:38:34]

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc
File cc/resources/picture_pile.cc (left):

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc...
cc/resources/picture_pile.cc:28: bool rect_sort_y(const gfx::Rect &r1, const
gfx::Rect &r2) {
This name is also correct.  No need to rename it.

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc
File cc/resources/picture_pile.cc (right):

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc...
cc/resources/picture_pile.cc:17: namespace cc {
No need for this.  These are all local to this file.

[hyunki, 2014-07-09 17:46:55]

Thanks. PTAL.

[hyunki, 2014-07-09 17:47:27]

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc
File cc/resources/picture_pile.cc (left):

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc...
cc/resources/picture_pile.cc:28: bool rect_sort_y(const gfx::Rect &r1, const
gfx::Rect &r2) {
On 2014/07/09 17:38:34, enne wrote:
> This name is also correct.  No need to rename it.

Done.

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc
File cc/resources/picture_pile.cc (right):

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc...
cc/resources/picture_pile.cc:17: namespace cc {
On 2014/07/09 17:38:34, enne wrote:
> No need for this.  These are all local to this file.

Done.

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc
File cc/resources/picture_pile.cc (left):

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc...
cc/resources/picture_pile.cc:28: bool rect_sort_y(const gfx::Rect &r1, const
gfx::Rect &r2) {
On 2014/07/09 17:38:34, enne wrote:
> This name is also correct.  No need to rename it.

Done.

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc
File cc/resources/picture_pile.cc (right):

https://codereview.chromium.org/375403003/diff/1/cc/resources/picture_pile.cc...
cc/resources/picture_pile.cc:17: namespace cc {
On 2014/07/09 17:38:34, enne wrote:
> No need for this.  These are all local to this file.

Done.

[hyunki, 2014-07-10 00:01:59]

Sorry. I have made duplicated replay for the comment. 
Ping PTAL. Thanks.

[enne (OOO), 2014-07-10 00:05:12]

Sure, lgtm.

[enne (OOO), 2014-07-10 00:05:15]

The CQ bit was checked by enne@chromium.org

[commit-bot: I haz the power, 2014-07-10 00:06:57]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hyunki.baik@samsung.com/375403003/20001

[commit-bot: I haz the power, 2014-07-10 03:47:47]

Change committed as 282244