New issue
Advanced search Search tips

Issue 637403 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Feature

Blocking:
issue 604105



Sign in to add a comment

Restructure ChooserContentViewCocoa's constructor

Project Member Reported by juncai@chromium.org, Aug 12 2016

Issue description

Restructure this function to be some smaller methods to create the pieces for the view. By doing so, the methods that calculate the frame and origins can be moved into those methods, rather than as helper functions.
 

Comment 1 by scheib@chromium.org, Oct 17 2016

Labels: Type-Feature

Comment 2 by scheib@chromium.org, Oct 20 2016

Blocking: 604105

Comment 3 by juncai@chromium.org, Dec 21 2016

This issue was created to address the comment at:
https://codereview.chromium.org/2208983002
Since now the calculateX functions (which were used to calculate the frame and origins) were removed, it seems this issue is already resolved.

Comment 4 by juncai@chromium.org, Dec 21 2016

Status: WontFix (was: Assigned)

Sign in to add a comment