[Reland][Extensions] Don't count bubble focus loss as acknowledgment
Currently, if an extension message bubble is shown, and then it is dismissed
because it loses focus, we treat it the same as the user clicking the dismiss
button - which serves as acknowledging the extension. We could ignore focus
loss, but this makes for very noisy, awkward bubbles. Instead, allow the bubble
to close, but don't treat this as user acknowledgment, and show the bubble again
on next startup.
This also involves tracking the close reason for a BubbleDelegateView.
BUG=548269
[email protected] (no changes relevant from original CL)
[email protected] (no changes relevant from original CL)
Review URL: https://codereview.chromium.org/1455313002
Cr-Commit-Position: refs/heads/master@{#360848}
diff --git a/chrome/browser/extensions/proxy_overridden_bubble_delegate.h b/chrome/browser/extensions/proxy_overridden_bubble_delegate.h
index cbd851d1..dedc843 100644
--- a/chrome/browser/extensions/proxy_overridden_bubble_delegate.h
+++ b/chrome/browser/extensions/proxy_overridden_bubble_delegate.h
@@ -40,6 +40,7 @@
bool ShouldLimitToEnabledExtensions() const override;
void LogExtensionCount(size_t count) override;
void LogAction(ExtensionMessageBubbleController::BubbleAction) override;
+ const char* GetKey() override;
private:
// The ID of the extension we are showing the bubble for.
