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

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

Issue 17382005: Unbreak tabs.onRemove extension API in face of fast tab closure Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't modify content Created 7 years, 6 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 | Annotate | Revision Log
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 477 matching lines...) Expand 10 before | Expand all | Expand 10 after
488 tab_strip_model_->RemoveObserver(this); 488 tab_strip_model_->RemoveObserver(this);
489 } 489 }
490 490
491 void TabCountChangeObserver::TabInsertedAt(WebContents* contents, 491 void TabCountChangeObserver::TabInsertedAt(WebContents* contents,
492 int index, 492 int index,
493 bool foreground) { 493 bool foreground) {
494 CheckTabCount(); 494 CheckTabCount();
495 } 495 }
496 496
497 void TabCountChangeObserver::TabDetachedAt(WebContents* contents, 497 void TabCountChangeObserver::TabDetachedAt(WebContents* contents,
498 int index) { 498 int index,
499 bool closing_all) {
499 CheckTabCount(); 500 CheckTabCount();
500 } 501 }
501 502
502 void TabCountChangeObserver::TabStripModelDeleted() { 503 void TabCountChangeObserver::TabStripModelDeleted() {
503 if (automation_.get()) { 504 if (automation_.get()) {
504 AutomationMsg_WaitForTabCountToBecome::WriteReplyParams( 505 AutomationMsg_WaitForTabCountToBecome::WriteReplyParams(
505 reply_message_.get(), false); 506 reply_message_.get(), false);
506 automation_->Send(reply_message_.release()); 507 automation_->Send(reply_message_.release());
507 } 508 }
508 509
(...skipping 2144 matching lines...) Expand 10 before | Expand all | Expand 10 after
2653 if (automation_.get()) { 2654 if (automation_.get()) {
2654 AutomationJSONReply(automation_.get(), reply_message_.release()) 2655 AutomationJSONReply(automation_.get(), reply_message_.release())
2655 .SendSuccess(NULL); 2656 .SendSuccess(NULL);
2656 } 2657 }
2657 delete this; 2658 delete this;
2658 } 2659 }
2659 } else { 2660 } else {
2660 NOTREACHED(); 2661 NOTREACHED();
2661 } 2662 }
2662 } 2663 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698