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

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: format fix. 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 722bf1a790899ee5a216ee902cf41ef357e27edc..c1eca42ed642188c26fb766da24de43b2da9530e 100644
--- a/cc/trees/layer_tree_host_unittest.cc
+++ b/cc/trees/layer_tree_host_unittest.cc
@@ -860,7 +860,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);
}
@@ -1903,7 +1903,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());
@@ -3296,11 +3296,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
@@ -4665,7 +4665,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();
@@ -4682,7 +4682,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();
@@ -4693,7 +4693,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();
@@ -4943,7 +4943,10 @@ 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) {}
protected:
enum { kExpectedNumCommits = 10 };

Powered by Google App Engine
This is Rietveld 408576698