Revert 239759 "The comment in base64.h implies that base::Base64..."

trunk/src/chrome/browser/ui/ash/screenshot_taker.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/ash/screenshot_taker.h"
 
 #include <climits>
 #include <string>
 
 #include "ash/shell.h"
@@ skipping 53 matching lines @@
 
 const char kNotificationOriginUrl[] = "chrome://screenshot";
 
 const char kImageClipboardFormatPrefix[] = "<img src='data:image/png;base64,";
 const char kImageClipboardFormatSuffix[] = "'>";
 
 void CopyScreenshotToClipboard(scoped_refptr<base::RefCountedString> png_data) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   std::string encoded;
-  base::Base64Encode(png_data->data(), &encoded);
+  if (!base::Base64Encode(png_data->data(), &encoded)) {
+    LOG(ERROR) << "Failed to encode base64";
+    return;
+  }
 
   // Only cares about HTML because ChromeOS doesn't need other formats.
   // TODO(dcheng): Why don't we take advantage of the ability to write bitmaps
   // to the clipboard here?
   {
     ui::ScopedClipboardWriter scw(ui::Clipboard::GetForCurrentThread(),
                                   ui::CLIPBOARD_TYPE_COPY_PASTE);
     std::string html(kImageClipboardFormatPrefix);
     html += encoded;
     html += kImageClipboardFormatSuffix;
@@ skipping 505 matching lines @@
 }
 
 void ScreenshotTaker::SetScreenshotBasenameForTest(
     const std::string& basename) {
   screenshot_basename_for_test_ = basename;
 }
 
 void ScreenshotTaker::SetScreenshotProfileForTest(Profile* profile) {
   profile_for_test_ = profile;
 }