Skip to content

Commit

Permalink
Fix master debug build failure from BraveActionAPIDependencyManager
Browse files Browse the repository at this point in the history
There was a missing abstract method that we missed only for debug build.
  • Loading branch information
simonhong committed Oct 27, 2019
1 parent 7dcb39c commit baf343c
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions browser/extensions/api/brave_action_api.cc
Original file line number Diff line number Diff line change
Expand Up @@ -29,14 +29,17 @@ class BraveActionAPIDependencyManager : public DependencyManager {
static base::NoDestructor<BraveActionAPIDependencyManager> factory;
return factory.get();
}
BraveActionAPIDependencyManager() { }
~BraveActionAPIDependencyManager() override;
BraveActionAPIDependencyManager() {}
~BraveActionAPIDependencyManager() override {}

#ifndef NDEBUG
void DumpContextDependencies(void* context) const override {}
#endif // NDEBUG

private:
DISALLOW_COPY_AND_ASSIGN(BraveActionAPIDependencyManager);
};

BraveActionAPIDependencyManager::~BraveActionAPIDependencyManager() { }

class BraveActionAPIFactory : public KeyedServiceFactory {
public:
BraveActionAPIFactory() : KeyedServiceFactory("BraveActionAPI",
Expand Down

0 comments on commit baf343c

Please sign in to comment.