Remove an unnecessary pointer layer in idle timer.

base/idle_timer.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/idle_timer.h"
 
 // We may not want to port idle_timer to Linux, but we have implemented it
 // anyway.  Define this to 1 to enable the Linux idle timer and then add the
 // libs that need to be linked (Xss).
 #define ENABLE_XSS_SUPPORT 0
@@ skipping 44 matching lines @@
   return true;
 }
 #elif defined(OS_LINUX) && ENABLE_XSS_SUPPORT
 class IdleState {
  public:
   IdleState() {
     int event_base, error_base;
     have_idle_info_ = XScreenSaverQueryExtension(GDK_DISPLAY(), &event_base,
                                                  &error_base);
     if (have_idle_info_)
-      *idle_info_.Get() = XScreenSaverAllocInfo();
+      idle_info_.Set(XScreenSaverAllocInfo());
   }
 
   ~IdleState() {
-    if (*idle_info_.Get()) {
-      XFree(*idle_info_.Get());
+    if (idle_info_.Get()) {
+      XFree(idle_info_.Get());
       idle_info_.~ThreadLocalPointer();
     }
   }
 
   int32 IdleTime() {
     if (have_idle_info_ && idle_info_.Get()) {
       XScreenSaverQueryInfo(GDK_DISPLAY(), GDK_ROOT_WINDOW(),
-                            *idle_info_.Get());
-      return (*idle_info_.Get())->idle;
+                            idle_info_.Get());
+      return idle_info_.Get()->idle;
     }
     return -1;
   }
 
  private:
   bool have_idle_info_;
-  ThreadLocalPointer<XScreenSaverInfo*> idle_info_;
+  ThreadLocalPointer<XScreenSaverInfo> idle_info_;
 
   DISALLOW_COPY_AND_ASSIGN(IdleState);
 };
 
 bool OSIdleTimeSource(int32* milliseconds_interval_since_last_event) {
   static LazyInstance<IdleState> state_instance(base::LINKER_INITIALIZED);
   IdleState* state = state_instance.Pointer();
   int32 idle_time = state->IdleTime();
   if (0 < idle_time) {
     *milliseconds_interval_since_last_event = idle_time;
@@ skipping 55 matching lines @@
   TimeDelta current_idle_time = CurrentIdleTime();
   if (current_idle_time > time_since_last_fire) {
     if (repeat_)
       return idle_interval_ - time_since_last_fire;
     return idle_interval_;
   }
   return idle_interval_ - current_idle_time;
 }
 
 }  // namespace base