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

Side by Side Diff: extensions/browser/extension_function.h

Issue 2118473002: 😴 Skip some KeepAlive registrations on shutdown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments Created 4 years, 5 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #ifndef EXTENSIONS_BROWSER_EXTENSION_FUNCTION_H_ 5 #ifndef EXTENSIONS_BROWSER_EXTENSION_FUNCTION_H_
6 #define EXTENSIONS_BROWSER_EXTENSION_FUNCTION_H_ 6 #define EXTENSIONS_BROWSER_EXTENSION_FUNCTION_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <list> 10 #include <list>
(...skipping 475 matching lines...) Expand 10 before | Expand all | Expand 10 after
486 public: 486 public:
487 virtual void OnSendResponse(UIThreadExtensionFunction* function, 487 virtual void OnSendResponse(UIThreadExtensionFunction* function,
488 bool success, 488 bool success,
489 bool bad_message) = 0; 489 bool bad_message) = 0;
490 }; 490 };
491 491
492 UIThreadExtensionFunction(); 492 UIThreadExtensionFunction();
493 493
494 UIThreadExtensionFunction* AsUIThreadExtensionFunction() override; 494 UIThreadExtensionFunction* AsUIThreadExtensionFunction() override;
495 495
496 bool PreRunValidation(std::string* error) override;
497
496 void set_test_delegate(DelegateForTests* delegate) { 498 void set_test_delegate(DelegateForTests* delegate) {
497 delegate_ = delegate; 499 delegate_ = delegate;
498 } 500 }
499 501
500 // Called when a message was received. 502 // Called when a message was received.
501 // Should return true if it processed the message. 503 // Should return true if it processed the message.
502 virtual bool OnMessageReceived(const IPC::Message& message); 504 virtual bool OnMessageReceived(const IPC::Message& message);
503 505
504 // Set the browser context which contains the extension that has originated 506 // Set the browser context which contains the extension that has originated
505 // this function call. 507 // this function call.
(...skipping 212 matching lines...) Expand 10 before | Expand all | Expand 10 after
718 private: 720 private:
719 // If you're hitting a compile error here due to "final" - great! You're 721 // If you're hitting a compile error here due to "final" - great! You're
720 // doing the right thing, you just need to extend IOThreadExtensionFunction 722 // doing the right thing, you just need to extend IOThreadExtensionFunction
721 // instead of SyncIOExtensionFunction. 723 // instead of SyncIOExtensionFunction.
722 ResponseAction Run() final; 724 ResponseAction Run() final;
723 725
724 DISALLOW_COPY_AND_ASSIGN(SyncIOThreadExtensionFunction); 726 DISALLOW_COPY_AND_ASSIGN(SyncIOThreadExtensionFunction);
725 }; 727 };
726 728
727 #endif // EXTENSIONS_BROWSER_EXTENSION_FUNCTION_H_ 729 #endif // EXTENSIONS_BROWSER_EXTENSION_FUNCTION_H_
OLDNEW
« no previous file with comments | « extensions/browser/api/guest_view/web_view/web_view_internal_api.cc ('k') | extensions/browser/extension_function.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698