Index: chrome/browser/ui/webui/feedback_ui.cc |
diff --git a/chrome/browser/ui/webui/feedback_ui.cc b/chrome/browser/ui/webui/feedback_ui.cc |
index bc994feca853c1dd2560b4407b18959af812ff65..812207484d4751a6d40cd8ce016bcf7d129046cf 100644 |
--- a/chrome/browser/ui/webui/feedback_ui.cc |
+++ b/chrome/browser/ui/webui/feedback_ui.cc |
@@ -13,12 +13,14 @@ |
#include "base/message_loop.h" |
#include "base/string_number_conversions.h" |
#include "base/string_util.h" |
-#include "base/utf_string_conversions.h" |
#include "base/time.h" |
+#include "base/utf_string_conversions.h" |
#include "base/values.h" |
+#include "chrome/browser/browser_process.h" |
#include "chrome/browser/download/download_prefs.h" |
#include "chrome/browser/feedback/feedback_data.h" |
#include "chrome/browser/feedback/feedback_util.h" |
+#include "chrome/browser/prefs/pref_service.h" |
#include "chrome/browser/profiles/profile.h" |
#include "chrome/browser/ui/browser.h" |
#include "chrome/browser/ui/browser_finder.h" |
@@ -31,6 +33,7 @@ |
#include "chrome/browser/ui/webui/screenshot_source.h" |
#include "chrome/browser/ui/window_snapshot/window_snapshot.h" |
#include "chrome/common/chrome_paths.h" |
+#include "chrome/common/pref_names.h" |
#include "chrome/common/url_constants.h" |
#include "content/public/browser/browser_thread.h" |
#include "content/public/browser/navigation_controller.h" |
@@ -168,9 +171,9 @@ void ShowWebFeedbackView(Browser* browser, |
native_window = browser->window()->GetNativeWindow(); |
snapshot_bounds = gfx::Rect(browser->window()->GetBounds().size()); |
#endif |
- bool success = browser::GrabWindowSnapshot(native_window, |
- last_screenshot_png, |
- snapshot_bounds); |
+ bool success = browser::GrabWindowSnapshotIfPermitted(native_window, |
+ last_screenshot_png, |
+ snapshot_bounds); |
FeedbackUtil::SetScreenshotSize(success ? snapshot_bounds : gfx::Rect()); |
std::string feedback_url = std::string(chrome::kChromeUIFeedbackURL) + "?" + |
@@ -447,9 +450,12 @@ void FeedbackHandler::HandleGetDialogDefaults(const ListValue*) { |
else |
dialog_defaults.Append(new StringValue("")); |
+ // 1: are screenshots disabled? |
+ bool disable_screenshots = |
+ g_browser_process->local_state()->GetBoolean(prefs::kDisableScreenshots); |
+ dialog_defaults.Append(base::Value::CreateBooleanValue(disable_screenshots)); |
+ |
#if defined(OS_CHROMEOS) |
- // 1: about:system |
- dialog_defaults.Append(new StringValue(chrome::kChromeUISystemInfoURL)); |
// Trigger the request for system information here. |
chromeos::system::SyslogsProvider* provider = |
chromeos::system::SyslogsProvider::GetInstance(); |