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

Side by Side Diff: chrome/browser/automation/automation_provider_observers.cc

Issue 10113005: Remove EPM:all_hosts_ and use all_extension_views_ instead. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: interactive_ui_tests 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/automation/automation_provider_observers.h" 5 #include "chrome/browser/automation/automation_provider_observers.h"
6 6
7 #include <deque> 7 #include <deque>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 489 matching lines...) Expand 10 before | Expand all | Expand 10 after
500 if (automation_) { 500 if (automation_) {
501 AutomationMsg_WaitForTabCountToBecome::WriteReplyParams( 501 AutomationMsg_WaitForTabCountToBecome::WriteReplyParams(
502 reply_message_.get(), true); 502 reply_message_.get(), true);
503 automation_->Send(reply_message_.release()); 503 automation_->Send(reply_message_.release());
504 } 504 }
505 505
506 delete this; 506 delete this;
507 } 507 }
508 508
509 bool DidExtensionHostsStopLoading(ExtensionProcessManager* manager) { 509 bool DidExtensionHostsStopLoading(ExtensionProcessManager* manager) {
510 for (ExtensionProcessManager::const_iterator iter = manager->begin(); 510 for (ExtensionProcessManager::const_iterator iter =
511 iter != manager->end(); ++iter) { 511 manager->background_hosts().begin();
512 iter != manager->background_hosts().end(); ++iter) {
512 if (!(*iter)->did_stop_loading()) 513 if (!(*iter)->did_stop_loading())
513 return false; 514 return false;
514 } 515 }
515 return true; 516 return true;
516 } 517 }
517 518
518 ExtensionUninstallObserver::ExtensionUninstallObserver( 519 ExtensionUninstallObserver::ExtensionUninstallObserver(
519 AutomationProvider* automation, 520 AutomationProvider* automation,
520 IPC::Message* reply_message, 521 IPC::Message* reply_message,
521 const std::string& id) 522 const std::string& id)
(...skipping 2531 matching lines...) Expand 10 before | Expand all | Expand 10 after
3053 } 3054 }
3054 3055
3055 ExtensionHost* host = content::Details<ExtensionHost>(details).ptr(); 3056 ExtensionHost* host = content::Details<ExtensionHost>(details).ptr();
3056 if (host->extension_id() == extension_id_ && 3057 if (host->extension_id() == extension_id_ &&
3057 host->extension_host_type() == chrome::VIEW_TYPE_EXTENSION_POPUP) { 3058 host->extension_host_type() == chrome::VIEW_TYPE_EXTENSION_POPUP) {
3058 AutomationJSONReply(automation_, reply_message_.release()) 3059 AutomationJSONReply(automation_, reply_message_.release())
3059 .SendSuccess(NULL); 3060 .SendSuccess(NULL);
3060 delete this; 3061 delete this;
3061 } 3062 }
3062 } 3063 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/automation/automation_util.h » ('j') | chrome/browser/automation/automation_util.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698