Skip to content
This repository has been archived by the owner on Jan 4, 2019. It is now read-only.

Implement subclass of SSLCertificateViewerMac #175

Closed
darkdh opened this issue Apr 5, 2017 · 1 comment
Closed

Implement subclass of SSLCertificateViewerMac #175

darkdh opened this issue Apr 5, 2017 · 1 comment
Assignees

Comments

@darkdh
Copy link
Member

darkdh commented Apr 5, 2017

We currently new a instance of this base class. It will cause a crash from Debug build

- (void)sheetDidEnd:(NSWindow*)parent
         returnCode:(NSInteger)returnCode
            context:(void*)context {
  NOTREACHED();  // Subclasses must implement this.
}

in chrome/browser/ui/certificate_viewer_mac.mm

reference
chrome/browser/ui/cocoa/certificate_viewer_mac_cocoa.h and chrome/browser/ui/cocoa/certificate_viewer_mac_cocoa.mm

@darkdh darkdh self-assigned this Apr 5, 2017
@darkdh
Copy link
Member Author

darkdh commented May 9, 2017

fixed by d534ffd

@darkdh darkdh closed this as completed May 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant