• sadrul@chromium.org's avatar
    Revert 291286: broke 'ExtensionOptionsApiTest.ExtensionCanEmbedOwnOptions'. · 9cd23e39
    sadrul@chromium.org authored
    The test is failing on all platforms with the following error message:
    "[FAIL] createGuestViewDOM: uncaught exception: Error in event handler for extensionOptionsInternal.onSizeChanged: Invocation of form guestViewInternal.setAutoSize(undefined, object) doesn't match definition guestViewInternal.setAutoSize(integer instanceId, object params)
    Stack trace: Error: Invocation of form guestViewInternal.setAutoSize(undefined, object) doesn't match definition guestViewInternal.setAutoSize(integer instanceId, object params)
        at Object.normalizeArgumentsAndValidate (extensions::schemaUtils:113:11)
        at Object.\u003Canonymous> (extensions::binding:326:30)
        at ExtensionOptionsInternal.resize (extensions::extensionOptions:178:21)
        at ExtensionOptionsInternal.onSizeChanged (extensions::extensionOptions:153:10)
        at ExtensionOptionsEvents.handleSizeChangedEvent (extensions::extensionOptionsEvents:66:33)
        at Object.EXTENSION_OPTIONS_EVENTS.sizechanged.customHandler (extensions::extensionOptionsEvents:20:15)
        at info.evt.addListener.instanceId (extensions::extensionOptionsEvents:55:12)
        at EventImpl.dispatchToListener (extensions::event_bindings:397:22)
        at Event.publicClass.(anonymous function) [as dispatchToListener] (extensions::utils:94:26)
        at EventImpl.dispatch_ (extensions::event_bindings:379:35)
    Error
        at extensions::test:93:22
        at exports.handle (extensions::uncaught_exception_handler:15:3)
        at EventImpl.dispatch_ (extensions::event_bindings:384:9)
        at dispatchArgs (extensions::event_bindings:247:26)
        at dispatchEvent (extensions::event_bindings:256:7)", source: chrome-extension://obocclgnomgjhjnenilbpmgkgfmdhecn/test.html (0)
    
    > Implement smoother autosizing of the extension options overlay
    > 
    >  - Use element.animate to resize the overlay for embedded extension
    >    options. When an <extensionoptions> element is embedded in a WebUI
    >    overlay, if its size changes the overlay will smoothly expand to
    >    surround the element.
    >  - Add an API to <extensionoptions> to defer autosizing until its
    >    embedder is ready. This is used to 'pin' the element's size until
    >    the overlay is done expanding.
    >  - Move the <extensionoptions> off screen until the overlay is done
    >    expanding.
    >  - Prevent <extensionoptions> from shrinking after the initial sizing.
    >    This makes the look of the embedded options page more consistent.
    > 
    > BUG=386842
    > 
    > Review URL: https://codereview.chromium.org/480243003
    
    TBR=ericzeng@chromium.org
    
    Review URL: https://codereview.chromium.org/498633002
    
    Cr-Commit-Position: refs/heads/master@{#291328}
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291328 0039d316-1c4b-4281-b951-d872f2087c98
    9cd23e39
extensions.html 10.4 KB