Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(312)

Unified Diff: chrome/browser/automation/automation_provider_observers.cc

Issue 10033001: Convert printing ui_tests to browser_tests. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/automation/automation_provider_observers.cc
===================================================================
--- chrome/browser/automation/automation_provider_observers.cc (revision 131326)
+++ chrome/browser/automation/automation_provider_observers.cc (working copy)
@@ -44,7 +44,6 @@
#include "chrome/browser/notifications/notification_ui_manager.h"
#include "chrome/browser/password_manager/password_store_change.h"
#include "chrome/browser/policy/browser_policy_connector.h"
-#include "chrome/browser/printing/print_job.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/renderer_host/chrome_render_message_filter.h"
#include "chrome/browser/search_engines/template_url_service.h"
@@ -1282,57 +1281,6 @@
}
}
-DocumentPrintedNotificationObserver::DocumentPrintedNotificationObserver(
Lei Zhang 2012/04/09 19:44:44 Remove from automation_provider_observers.h as wel
jam 2012/04/09 20:16:22 oops, forgot to include that file in my change :)
- AutomationProvider* automation, IPC::Message* reply_message)
- : automation_(automation->AsWeakPtr()),
- success_(false),
- reply_message_(reply_message) {
- registrar_.Add(this, chrome::NOTIFICATION_PRINT_JOB_EVENT,
- content::NotificationService::AllSources());
-}
-
-DocumentPrintedNotificationObserver::~DocumentPrintedNotificationObserver() {
- if (automation_) {
- AutomationMsg_PrintNow::WriteReplyParams(reply_message_.get(), success_);
- automation_->Send(reply_message_.release());
- }
-}
-
-void DocumentPrintedNotificationObserver::Observe(
- int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) {
- DCHECK(type == chrome::NOTIFICATION_PRINT_JOB_EVENT);
- switch (content::Details<printing::JobEventDetails>(details)->type()) {
- case printing::JobEventDetails::JOB_DONE: {
- // Succeeded.
- success_ = true;
- delete this;
- break;
- }
- case printing::JobEventDetails::USER_INIT_CANCELED:
- case printing::JobEventDetails::FAILED: {
- // Failed.
- delete this;
- break;
- }
- case printing::JobEventDetails::NEW_DOC:
- case printing::JobEventDetails::USER_INIT_DONE:
- case printing::JobEventDetails::DEFAULT_INIT_DONE:
- case printing::JobEventDetails::NEW_PAGE:
- case printing::JobEventDetails::PAGE_DONE:
- case printing::JobEventDetails::DOC_DONE:
- case printing::JobEventDetails::ALL_PAGES_REQUESTED: {
- // Don't care.
- break;
- }
- default: {
- NOTREACHED();
- break;
- }
- }
-}
-
MetricEventDurationObserver::MetricEventDurationObserver() {
registrar_.Add(this, chrome::NOTIFICATION_METRIC_EVENT_DURATION,
content::NotificationService::AllSources());

Powered by Google App Engine
This is Rietveld 408576698