screenshot disabling policy

chrome/browser/ui/views/ash/chrome_shell_delegate.cc

 // Copyright (c) 2012 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/ash/chrome_shell_delegate.h"
 
 #include "ash/launcher/launcher_types.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/wm/partial_screenshot_view.h"
 #include "ash/wm/window_util.h"
@@ skipping 16 matching lines @@
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
 #include "grit/generated_resources.h"
 #include "ui/aura/client/user_action_client.h"
 #include "ui/aura/window.h"
 
 #if defined(OS_CHROMEOS)
+#include "ash/accelerators/accelerator_controller.h"
 #include "base/chromeos/chromeos_version.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/accessibility/accessibility_util.h"
 #include "chrome/browser/chromeos/background/ash_user_wallpaper_delegate.h"
 #include "chrome/browser/chromeos/extensions/file_manager_util.h"
 #include "chrome/browser/chromeos/kiosk_mode/kiosk_mode_settings.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/login/webui_login_display_host.h"
 #include "chrome/browser/chromeos/system/ash_system_tray_delegate.h"
+#include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/ui/views/ash/screenshot_taker.h"
 #include "chrome/browser/ui/views/keyboard_overlay_dialog_view.h"
 #include "chrome/browser/ui/webui/chromeos/login/oobe_ui.h"
 #include "chrome/browser/ui/webui/chromeos/mobile_setup_dialog.h"
+#include "chrome/common/pref_names.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/power_manager_client.h"
+#include "content/public/browser/browser_thread.h"
 #endif
 
 namespace {
 
 // Returns the browser that should handle accelerators.
 Browser* GetTargetBrowser() {
   Browser* browser = browser::FindBrowserWithWindow(ash::wm::GetActiveWindow());
   if (browser)
     return browser;
   return browser::FindOrCreateTabbedBrowser(
       ProfileManager::GetDefaultProfileOrOffTheRecord());
 }
 
 }  // namespace
 
 // static
 ChromeShellDelegate* ChromeShellDelegate::instance_ = NULL;
 
 ChromeShellDelegate::ChromeShellDelegate()
     : window_positioner_(new WindowPositioner()),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) {
   instance_ = this;
 #if defined(OS_CHROMEOS)
   registrar_.Add(
       this,
       chrome::NOTIFICATION_LOGIN_USER_PROFILE_PREPARED,
       content::NotificationService::AllSources());
+  pref_registrar_.Init(g_browser_process->local_state());
+  pref_registrar_.Add(prefs::kDisableScreenshots, this);
 #endif
 }
 
 ChromeShellDelegate::~ChromeShellDelegate() {
   if (instance_ == this)
     instance_ = NULL;
 }
 
 bool ChromeShellDelegate::IsUserLoggedIn() {
 #if defined(OS_CHROMEOS)
@@ skipping 237 matching lines @@
 }
 
 void ChromeShellDelegate::Observe(int type,
                                   const content::NotificationSource& source,
                                   const content::NotificationDetails& details) {
 #if defined(OS_CHROMEOS)
   switch (type) {
     case chrome::NOTIFICATION_LOGIN_USER_PROFILE_PREPARED:
       ash::Shell::GetInstance()->CreateLauncher();
       break;
+    case chrome::NOTIFICATION_PREF_CHANGED:
+      DCHECK(prefs::kDisableScreenshots ==

[Mattias Nissler (ping if slow), 2012/07/06 15:15:52]

DCHECK_EQ

[qfel, 2012/07/06 17:00:39]

Done

+        *content::Details<std::string>(details).ptr());

[Mattias Nissler (ping if slow), 2012/07/06 15:15:52]

4 spaces indentation after line breaks

[qfel, 2012/07/06 17:00:39]

Done

+      UpdateScreenshotDelegate();
+      break;
     default:
       NOTREACHED() << "Unexpected notification " << type;
   }
 #else
   // MSVC++ warns about switch statements without any cases.
   NOTREACHED() << "Unexpected notification " << type;
 #endif
 }
+
+#if defined(OS_CHROMEOS)
+void ChromeShellDelegate::UpdateScreenshotDelegate()
+{
+  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));

[Mattias Nissler (ping if slow), 2012/07/06 15:15:52]

I don't think we need this here, because the class lives on the UI thread.

[qfel, 2012/07/06 17:00:39]

Done

+  ash::AcceleratorController* accelerator_controller =
+    ash::Shell::GetInstance()->accelerator_controller();

[Mattias Nissler (ping if slow), 2012/07/06 15:15:52]

indentation

[qfel, 2012/07/06 17:00:39]

Done

+  PrefService* service = g_browser_process->local_state();
+  DCHECK(service);
+  if (service->GetBoolean(prefs::kDisableScreenshots))

[Mattias Nissler (ping if slow), 2012/07/06 15:15:52]

add curlies for multi-line block

[qfel, 2012/07/06 17:00:39]

Done.

+    accelerator_controller->SetScreenshotDelegate(
+      scoped_ptr<ash::ScreenshotDelegate>());
+  else
+    accelerator_controller->SetScreenshotDelegate(
+      scoped_ptr<ash::ScreenshotDelegate>(new ScreenshotTaker).Pass());

[Mattias Nissler (ping if slow), 2012/07/06 15:15:52]

Maybe this is shorter:

scoped_ptr<ash::ScreenshotDelegate> delegate;
if (/* enabled */)
    delegate.reset(new ScreenshotTaker);
accelerator_controller->SetScreenshotDelegate(delegate.Pass());

[qfel, 2012/07/06 17:00:39]

Done.

+}
+#endif