Skip to content

Commit

Permalink
ash static initializers: Remove the final two static initializers.
Browse files Browse the repository at this point in the history
ash::DisplayController was still depending on Display::kInvalidDisplayID
in a way that emitted static initializers. Just initialize it to -1;
DisplayController::DisplayController() still sets the value during its
constructor.

Remove one more instance of <iostream>. I was confused by the output
because there were actually two different files called debug_utils.cc in
the ash build which weren't in the normal linux build which both were
including <iostream>.

BUG=300084
R=sky@chromium.org

Review URL: https://codereview.chromium.org/272133003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270472 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
erg@chromium.org committed May 14, 2014
1 parent 9bd0b17 commit 85796b2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion ash/display/display_controller.cc
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@ namespace {
// accessed after Shell is deleted. A separate display instance is created
// during the shutdown instead of always keeping two display instances
// (one here and another one in display_manager) in sync, which is error prone.
int64 primary_display_id = gfx::Display::kInvalidDisplayID;
// This is initialized in the constructor, and then in CreatePrimaryHost().
int64 primary_display_id = -1;

// Specifies how long the display change should have been disabled
// after each display change operations.
Expand Down
2 changes: 1 addition & 1 deletion ui/compositor/debug_utils.cc
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

#include <cmath>
#include <iomanip>
#include <iostream>
#include <ostream>
#include <string>

#include "base/logging.h"
Expand Down

0 comments on commit 85796b2

Please sign in to comment.