Expose default ctor of SQS and remove Create and Copy funciton of SQS

Expose default ctor of SQS and remove Create and Copy funciton of SQS

In order to avoid passing around ownership of SQS, expose default ctor,
and remove all function that returns scoped_ptr of SQS, meaning Create
and Copy function. Alternative of Copy function is CopyFrom function
that is called on raw pointer.

This follows 265823015 and 271703003.

BUG=344962
TEST=cc_unittests

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

[danakj, 2014-05-08 16:51:41]

LGTM

https://codereview.chromium.org/271683004/diff/1/cc/quads/draw_quad_unittest.cc
File cc/quads/draw_quad_unittest.cc (right):

https://codereview.chromium.org/271683004/diff/1/cc/quads/draw_quad_unittest....
cc/quads/draw_quad_unittest.cc:42: scoped_ptr<SharedQuadState> state =
make_scoped_ptr(new SharedQuadState());
btw this also works

scoped_ptr<SQS> state(new SQS);

https://codereview.chromium.org/271683004/diff/1/cc/quads/render_pass.cc
File cc/quads/render_pass.cc (right):

https://codereview.chromium.org/271683004/diff/1/cc/quads/render_pass.cc#newc...
cc/quads/render_pass.cc:173:
shared_quad_state_list.push_back(make_scoped_ptr(new SharedQuadState()));
nit: no () for new with default constructor

[weiliangc, 2014-05-12 21:54:53]

Updated addressing review comments and rebase.

https://codereview.chromium.org/271683004/diff/1/cc/quads/draw_quad_unittest.cc
File cc/quads/draw_quad_unittest.cc (right):

https://codereview.chromium.org/271683004/diff/1/cc/quads/draw_quad_unittest....
cc/quads/draw_quad_unittest.cc:42: scoped_ptr<SharedQuadState> state =
make_scoped_ptr(new SharedQuadState());
On 2014/05/08 16:51:41, danakj wrote:
> btw this also works
> 
> scoped_ptr<SQS> state(new SQS);

Done.

https://codereview.chromium.org/271683004/diff/1/cc/quads/render_pass.cc
File cc/quads/render_pass.cc (right):

https://codereview.chromium.org/271683004/diff/1/cc/quads/render_pass.cc#newc...
cc/quads/render_pass.cc:173:
shared_quad_state_list.push_back(make_scoped_ptr(new SharedQuadState()));
On 2014/05/08 16:51:41, danakj wrote:
> nit: no () for new with default constructor

Done.

[weiliangc, 2014-05-12 22:12:02]

The CQ bit was checked by weiliangc@chromium.org

[commit-bot: I haz the power, 2014-05-12 22:13:04]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/weiliangc@chromium.org/271683004/40001

[commit-bot: I haz the power, 2014-05-12 22:50:52]

Change committed as 269901