• ccameron's avatar
    Revert "MacPWAs: Reparent tabs when installing PWAs" · 2f3e9151
    ccameron authored
    This reverts commit 96b864cc.
    
    Reason for revert: This sometimes results in mangled .app bundles, needs a cleaner solution.
    
    Original change's description:
    > MacPWAs: Reparent tabs when installing PWAs
    > 
    > This works now. When re-parenting the window, the initial attempt to
    > re-parent the window fails, because the PWA's bundle has not yet
    > been created. The second attempt then re-creates the bundle and
    > succeeds. This is a bit kludgey in that it is possible that the bundle
    > will be recreated twice.
    > 
    > A cleaner solution would be to delay reparenting the contents and
    > revealing the app in Finder until the app has been created, but that
    > solution may not be suitable for merge.
    > 
    > Bug: 859152
    > Change-Id: Ifaf507525729a9a168e821f722aee03e9f6dd8f4
    > Reviewed-on: https://chromium-review.googlesource.com/c/1438037
    > Reviewed-by: Ben Wells <benwells@chromium.org>
    > Reviewed-by: Dominick Ng <dominickn@chromium.org>
    > Commit-Queue: ccameron <ccameron@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#627151}
    
    TBR=benwells@chromium.org,ccameron@chromium.org,dominickn@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 859152
    Change-Id: I6460da08a6dfb8f3e68218ecc19216773062458a
    Reviewed-on: https://chromium-review.googlesource.com/c/1450216Reviewed-by: default avatarccameron <ccameron@chromium.org>
    Commit-Queue: ccameron <ccameron@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#628465}
    2f3e9151
bookmark_app_helper.cc 22.8 KB