Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3058)

Unified Diff: cc/trees/layer_tree_host_unittest.cc

Issue 640203002: [C++11 Allowed Features] Declares a type-safe null pointer converting from NULL to nullptr [part-… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/trees/layer_tree_host_unittest.cc
diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_unittest.cc
index 7add406971f35e80520b95d67541003b3e26ca51..f5fdb727347bcc99c25cc8b8377e98be6414fc18 100644
--- a/cc/trees/layer_tree_host_unittest.cc
+++ b/cc/trees/layer_tree_host_unittest.cc
@@ -857,7 +857,7 @@ class LayerTreeHostTestStartPageScaleAnimation : public LayerTreeHostTest {
// This test requires the page_scale and inner viewport layers to be
// identified.
layer_tree_host()->RegisterViewportLayers(
- root_layer, scroll_layer_.get(), NULL);
+ root_layer, scroll_layer_.get(), nullptr);
layer_tree_host()->SetPageScaleFactorAndLimits(1.f, 0.5f, 2.f);
}
@@ -1900,7 +1900,7 @@ class LayerTreeHostWithProxy : public LayerTreeHost {
LayerTreeHostWithProxy(FakeLayerTreeHostClient* client,
const LayerTreeSettings& settings,
scoped_ptr<FakeProxy> proxy)
- : LayerTreeHost(client, NULL, settings) {
+ : LayerTreeHost(client, nullptr, settings) {
proxy->SetLayerTreeHost(this);
client->SetLayerTreeHost(this);
InitializeForTesting(proxy.Pass());
@@ -3286,11 +3286,11 @@ class LayerTreeHostTestImplLayersPushProperties
break;
}
- PushPropertiesCountingLayerImpl* root_impl_ = NULL;
- PushPropertiesCountingLayerImpl* child_impl_ = NULL;
- PushPropertiesCountingLayerImpl* child2_impl_ = NULL;
- PushPropertiesCountingLayerImpl* grandchild_impl_ = NULL;
- PushPropertiesCountingLayerImpl* leaf_always_pushing_layer_impl_ = NULL;
+ PushPropertiesCountingLayerImpl* root_impl_ = nullptr;
+ PushPropertiesCountingLayerImpl* child_impl_ = nullptr;
+ PushPropertiesCountingLayerImpl* child2_impl_ = nullptr;
+ PushPropertiesCountingLayerImpl* grandchild_impl_ = nullptr;
+ PushPropertiesCountingLayerImpl* leaf_always_pushing_layer_impl_ = nullptr;
// Pull the layers that we need from the tree assuming the same structure
// as LayerTreeHostTestLayersPushProperties
@@ -4655,7 +4655,7 @@ class LayerTreeHostTestSimpleSwapPromiseMonitor : public LayerTreeHostTest {
{
scoped_ptr<SimpleSwapPromiseMonitor> swap_promise_monitor(
new SimpleSwapPromiseMonitor(layer_tree_host(),
- NULL,
+ nullptr,
&set_needs_commit_count,
&set_needs_redraw_count));
layer_tree_host()->SetNeedsCommit();
@@ -4672,7 +4672,7 @@ class LayerTreeHostTestSimpleSwapPromiseMonitor : public LayerTreeHostTest {
{
scoped_ptr<SimpleSwapPromiseMonitor> swap_promise_monitor(
new SimpleSwapPromiseMonitor(layer_tree_host(),
- NULL,
+ nullptr,
&set_needs_commit_count,
&set_needs_redraw_count));
layer_tree_host()->SetNeedsUpdateLayers();
@@ -4683,7 +4683,7 @@ class LayerTreeHostTestSimpleSwapPromiseMonitor : public LayerTreeHostTest {
{
scoped_ptr<SimpleSwapPromiseMonitor> swap_promise_monitor(
new SimpleSwapPromiseMonitor(layer_tree_host(),
- NULL,
+ nullptr,
&set_needs_commit_count,
&set_needs_redraw_count));
layer_tree_host()->SetNeedsAnimate();
@@ -4933,7 +4933,7 @@ MULTI_THREAD_TEST_F(LayerTreeHostTestGpuRasterizationForced);
class LayerTreeHostTestContinuousPainting : public LayerTreeHostTest {
public:
LayerTreeHostTestContinuousPainting()
- : num_commits_(0), num_draws_(0), bounds_(20, 20), child_layer_(NULL) {}
+ : num_commits_(0), num_draws_(0), bounds_(20, 20), child_layer_(nullptr) {}
danakj 2014/10/09 16:00:53 and here
protected:
enum { kExpectedNumCommits = 10 };

Powered by Google App Engine
This is Rietveld 408576698