Aura under Linux

chrome/browser/ui/views/bubble/bubble.cc

 // Copyright (c) 2011 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 "chrome/browser/ui/views/bubble/bubble.h"
 
 #include <vector>
 
 #include "chrome/browser/ui/views/bubble/border_contents.h"
 #include "chrome/common/chrome_notification_types.h"
@@ skipping 93 matching lines @@
 #endif
   }
 }
 
 void Bubble::AnimationProgressed(const ui::Animation* animation) {
   // Set the opacity for the main contents window.
   unsigned char opacity = static_cast<unsigned char>(
       animation_->GetCurrentValue() * 255);
 #if defined(USE_AURA)
   // TODO(beng):
+  (void)opacity;

[Ben Goodger (Google), 2011/09/16 01:11:18]

que?

[Emmanuel Saint-loubert-Bié, 2011/09/16 02:34:54]

without the (void) you get a warning that you have an unused variable and since
warning are treated as errors this is the way I handled those in a prior life.
Let me know if you handle this differently in Chrome.
(BTW this suggest that Windows does not have the same compilation flag enabled).

[Ben Goodger (Google), 2011/09/16 14:46:55]

Huh. OK.

   NOTIMPLEMENTED();
 #elif defined(OS_WIN)
   SetLayeredWindowAttributes(GetNativeView(), 0,
       static_cast<byte>(opacity), LWA_ALPHA);
   contents_->SchedulePaint();
 
   // Also fade in/out the bubble border window.
   border_->SetOpacity(opacity);
   border_->border_contents()->SchedulePaint();
 #else
@@ skipping 309 matching lines @@
     animation_->Hide();
 }
 
 bool Bubble::AcceleratorPressed(const views::Accelerator& accelerator) {
   if (!delegate_ || delegate_->CloseOnEscape()) {
     DoClose(true);
     return true;
   }
   return false;
 }