| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's already disabled, so this should have no observable effect.
The database code stays around until I've written a migration to delete the
webintents table.
The apps schema code stays around so that 'intent' blocks in apps are
silently ignored instead of causing an error.
BUG=173194
TBR=piman
Review URL: https://codereview.chromium.org/12225076
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181702 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL allows Terms of Service to be specified for a Public Account. The
terms are set by specifying a URL through policy. Chrome downloads the
terms and presents them to the user. Only if the user accepts the terms
can the Public Account be entered.
This CL also fixes the regression described in issue 174314.
TEST=Manual
BUG=chromium-os:32780, 174314
TBR=nkostylev,mnissler
Review URL: https://codereview.chromium.org/12221097
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181656 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL allows Terms of Service to be specified for a Public Account. The
terms are set by specifying a URL through policy. Chrome downloads the
terms and presents them to the user. Only if the user accepts the terms
can the Public Account be entered.
This CL also fixes the regression described in issue 174314.
TEST=Manual
BUG=chromium-os:32780,174314
Review URL: https://codereview.chromium.org/12218014
TBR=bartfab@chromium.org
Review URL: https://codereview.chromium.org/12213087
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181512 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL allows Terms of Service to be specified for a Public Account. The
terms are set by specifying a URL through policy. Chrome downloads the
terms and presents them to the user. Only if the user accepts the terms
can the Public Account be entered.
This CL also fixes the regression described in issue 174314.
TEST=Manual
BUG=chromium-os:32780,174314
Review URL: https://codereview.chromium.org/12218014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181499 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
changed.
This is the same behavior as currently for other content settings, though it will likely change soon (see discussion in bug).
Strings are in https://codereview.chromium.org/12218033/
BUG=172778
Review URL: https://codereview.chromium.org/12207011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181303 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Support for screen capturing using getUserMedia() was added
in r180502 (behind a flag), but there were no notifications
to let the user know that the screen is being captured. This
CL dds InfoBar that is shown whenever a page requests screen
capturing and lets the user approve or deny the request.
BUG=134249
TBR=brettw@chromium.org (gypi changes)
Review URL: https://chromiumcodereview.appspot.com/12022024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181112 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=161115
Review URL: https://chromiumcodereview.appspot.com/12090127
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181107 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the Mac version of this Views CL:
https://codereview.chromium.org/11896021/
Currently the one click signin confirmation UI is implemented as a
bubble. This CL adds a tab modal dialog version of that UI.
Code common to the dialog and bubble has been moved to a new OneClickSigninViewController class.
XIB Changes:
OneClickSigninBubble.xib:
* aligned the baseline of the text in the advanced link with the
text in the OK button.
* changed the file owner to be OneClickSigninViewController
OneClickSigninDialog.xib
* same as OneClickSigninBubble.xib except it uses constrained
window sytle controls
Screenshots:
Bubble: http://i.imgur.com/HmJqoQk.png
Dialog: http://i.imgur.com/gaY2cym.png
BUG=171330
TEST=
To show the bubble I performed the following steps;
- ran chrome with --use-client-login-signin-flow
- signed into gmail
- clicked "Ok Sync" in the infobar
Testing the dialog requires access to server side stuff that's not public yet. I asked rogerta@ for instructions.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181056
Review URL: https://codereview.chromium.org/12082088
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181071 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Mac: One click signin dialog
>
> This is the Mac version of this Views CL:
> https://codereview.chromium.org/11896021/
>
> Currently the one click signin confirmation UI is implemented as a
> bubble. This CL adds a tab modal dialog version of that UI.
>
> Code common to the dialog and bubble has been moved to a new OneClickSigninViewController class.
>
> XIB Changes:
> OneClickSigninBubble.xib:
> * aligned the baseline of the text in the advanced link with the
> text in the OK button.
> * changed the file owner to be OneClickSigninViewController
> OneClickSigninDialog.xib
> * same as OneClickSigninBubble.xib except it uses constrained
> window sytle controls
>
> Screenshots:
> Bubble: http://i.imgur.com/HmJqoQk.png
> Dialog: http://i.imgur.com/gaY2cym.png
>
> BUG=171330
> TEST=
> To show the bubble I performed the following steps;
> - ran chrome with --use-client-login-signin-flow
> - signed into gmail
> - clicked "Ok Sync" in the infobar
> Testing the dialog requires access to server side stuff that's not public yet. I asked rogerta@ for instructions.
>
> Review URL: https://codereview.chromium.org/12082088
TBR=sail@chromium.org
Review URL: https://codereview.chromium.org/12213050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181061 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the Mac version of this Views CL:
https://codereview.chromium.org/11896021/
Currently the one click signin confirmation UI is implemented as a
bubble. This CL adds a tab modal dialog version of that UI.
Code common to the dialog and bubble has been moved to a new OneClickSigninViewController class.
XIB Changes:
OneClickSigninBubble.xib:
* aligned the baseline of the text in the advanced link with the
text in the OK button.
* changed the file owner to be OneClickSigninViewController
OneClickSigninDialog.xib
* same as OneClickSigninBubble.xib except it uses constrained
window sytle controls
Screenshots:
Bubble: http://i.imgur.com/HmJqoQk.png
Dialog: http://i.imgur.com/gaY2cym.png
BUG=171330
TEST=
To show the bubble I performed the following steps;
- ran chrome with --use-client-login-signin-flow
- signed into gmail
- clicked "Ok Sync" in the infobar
Testing the dialog requires access to server side stuff that's not public yet. I asked rogerta@ for instructions.
Review URL: https://codereview.chromium.org/12082088
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181056 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
to a common place to avoid ifdefs.
BUG=none
TEST=none
R=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/12211044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181048 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Autocheckout flow, the browser should prompt the user with a bubble asking if they want to use Autocheckout.
BUG=172123
Review URL: https://chromiumcodereview.appspot.com/12091094
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181042 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
WebContentsModalDialogManager
This CL moves the Views web contents modal dialog code towards the
interface described in the associated bug. Similar changes for Gtk and
Cocoa will be made in follow-on CLs.
To that end, this CL contains the following detailed changes:
- Introduce NativeWebContentsModalDialogManager for providing
platform-specific UI functionality within WebContentsModalDialogManager, and
implement NativeWebContentsModalDialogManagerViews.
- Under Views, move to a model where WebContentsModalDialogManager listens for
closing events rather than requiring notifications from the widget. Continue
using the existing model on Gtk and Cocoa, pending refactoring there.
- Provide a mechanism for disabling mouse-driven dialog movement on Widget.
- Remove NativeConstrainedWindowDelegate and subclasses as they're no longer
necessary due to the above two items.
- Update tests to account for closing notification to
WebContentsModalDialogManager being done from the event loop, rather than
synchronously.
- Add temporary ConstrainedWindowViews factory function to minimize code churn
at construction sites during this refactoring.
The next planned steps in this work are:
- Refactor ConstrainedWindow subclasses under Gtk and Cocoa to listen for
closing events rather than requiring notifications from the widget.
- Refactor uses of WebContentsModalDialog interface to use platform-specific
equivalents in platform-specific code and, in platform-independent code,
invoke functionality via some other mechanism: platform_utils,
WebContentsModalDialogManager, ???. Remove WebContentsModalDialog interface.
- Generalize WebContentsModalDialogManagerDelegate positioning interface and
write WebContentsModalDialogManager::ShowDialog using the new interface.
BUG=157161
Review URL: https://chromiumcodereview.appspot.com/12045037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180904 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
R=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/12079017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180894 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
more places that need to be converted from USE_ASH/USE_AURA to
HostDesktopType, but this covers all the images.
BUG=171859
TEST=none
R=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/12093117
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180633 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=173010
TEST=pass following switches to chrome: --install-from-webstore=<app_id> --app-id=<app_id> --force-app-mode
Review URL: https://codereview.chromium.org/12086076
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180535 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
disable the test for now.)
> Revert 180055: Very likely caused http://crbug.com/173885
> > Moving chrome://tracing to content.
> >
> > BUG=169170
> > Review URL: https://codereview.chromium.org/12088099
>
> TBR=jam@chromium.org
> Review URL: https://codereview.chromium.org/12194007
TBR=thakis@chromium.org
Review URL: https://codereview.chromium.org/12197003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180285 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Moving chrome://tracing to content.
>
> BUG=169170
> Review URL: https://codereview.chromium.org/12088099
TBR=jam@chromium.org
Review URL: https://codereview.chromium.org/12194007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180282 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> views: Remove simple_message_box_win.cc in favor of simple_message_box_views.cc
>
> simple_message_box_win.cc is not be needed nor compatible with the desktop-aura
> transition. It also uses the win32 appearance not the new Chrome style
> appearance currently in development. So it's bad twice ;)
>
> Just kill it and that eases the work that will need to be done later.
>
> By enabling simple_message_box_views on Windows means that it will be used for all
> ShowMessageBox() dialogs instead of current Windows MessageBox.
>
> BUG=166075,54988
> R=ben@chromium.org
>
> Review URL: https://codereview.chromium.org/12042083
Reverting as causes crashes on windows. Specifically this code is
not necessarily executed on the UI thread and views is not designed to
be used on anything but the UI thread.
TBR=tfarina@chromium.org
BUG=173751
Review URL: https://codereview.chromium.org/12173002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180208 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
It has taken on some responsibility besides just creating dialogs. I'm also planning on adding an additional method soon for accepting/dismissing an open dialog.
BUG=none
TBR=joth
Review URL: https://codereview.chromium.org/12082123
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180194 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Once everything builds with that set, I'll switch the default
of that to off, then merge that into webkit and delete the
webkit pieces, and then I'll delete the chromium bits.
Since enable_web_intents is still 1, this change has no observable effect.
BUG=173194
TEST=build with GYP_DEFINES=enable_web_intents=0. No compile errors,
fewer linker errors.
TBR=jschuh
Review URL: https://codereview.chromium.org/12089102
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180186 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Looks like this wasn't deleted when the mac webintents UI went
away in r173274.
BUG=173194
Review URL: https://codereview.chromium.org/12094096
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180076 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=169170
Review URL: https://codereview.chromium.org/12088099
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180055 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
implementation details from the public API.
BUG=169170
Review URL: https://codereview.chromium.org/12153002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180040 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
One of the long term goals of the sync team has been to pull sync out of
chrome_dll and into its own component. This should result in faster link
times for component builds, and cleaner demarcation between sync code
and the rest of chrome.
This patch does the following:
- Splits off sync.gyp into gypi files for sync_core, sync_api, sync_internal_api,
sync_notifier and sync_proto.
- Audits the dependencies of various targets in sync.gyp, sync_tests.gyp, and other
chrome gyp files, and makes sure all dependencies are explicitly declared.
- Makes targets declared in gyp files outside sync.gyp directly depend on
sync.gyp:sync instead of inner sync targets.
- Implements two versions of the target 'sync.gyp:sync':
1) In static mode, the public 'sync' target has a target type of 'none',
and is composed of the static library targets 'sync_api', 'sync_core',
'sync_internal_api', 'sync_notifier', and 'sync_proto'.
2) In component mode, we build the public 'sync' target into a single shared
library, which includes the contents of sync_api.gypi, sync_core.gypi,
sync_internal_api.gypi, sync_notifier.gypi, and sync_proto.gypi.
TBR=akalin,robertshield,thakis
BUG=136928
TEST=Set GYP_DEFINES="component=shared_library" and build the 'all' target on all platforms.
Review URL: https://codereview.chromium.org/11412211
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180034 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mobile (Android and iOS).
Added a new page chrome://net-export to start/stop/send the net_log
for mobile platforms. net_log entries are saved in
"chrome-net-export-log.json" file created in temporary directory.
Button "Start Saving (after deleting old data)" will start saving to a
temporary file in "NetLog" directory in Temp directory. It deletes all
the old temporary files that were in that directory before creating a
new temporty file. It is a no-op if we are already saving data into a
file.
Button "Stop Saving Data" closes the temporary file so that it could
be mailed. It is a no-op if we haven't started saving data into a
file.
Button "Send Data" emails the data file if a temporary file exists in
"NetLog" directory. It is a no-op if there is no file.
There is only a single temporary file into which we log the
data. NetLog data is saved in the browser process. Logging of net_log
data behaves similar to the command line option --log-net-log.
If chrome://net-export is accessed in multiple tabs, action in first
tab wins. If Stop is clicked in any tab, logging stops. If start is
clicked in any tab, net logging starts.
about:about lists chrome://net-export as one of the links.
unit tests create a temporary file for each test run and they delete the
file at the end of the test run.
mmenke@ and/or eroman@ for chrome\browser\resources and webui changes.
jar@ for net_log_temp_file changes.
jhawkins@chromium.org for chrome\browser\ui\webui and chrome\browser\resources
OWNERS stamp.
R=jar@chromium.org, mmenke@chromium.org
BUG=151212
TEST=chrome://net-export
Review URL: https://chromiumcodereview.appspot.com/11828036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180026 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=169170
Review URL: https://codereview.chromium.org/12086079
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179908 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the Mac verison of this Views CL:
https://chromiumcodereview.appspot.com/11418229/
This CL adds a search token decoration to the omnibox.
The search token is displayed when the user does a search from the omnibox.
The search token appears on the right with a separtor between it and other decorations.
Screenshot:
no separator: http://i.imgur.com/TXCixtn.png
with separator 1x: http://imgur.com/xtxv9hG
with separator 2x: http://imgur.com/2Pzi7dN
BUG=163190
TBR=thakis@chromium.org
Review URL: https://chromiumcodereview.appspot.com/12042002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179818 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This takes the existing message center code for Ash, and refactors it
into two layers - a system-specific layer (WebNotificationTray) and a
platform-independent (modulo Views) layer (MessageCenterTray). The
WebNotificationTray is responsible for rendering the tray icon and
noticing system changes that cause differences in rendering. The
MessageCenterTray delegates responsibility for rendering the message
center and notification bubbles to the WebNotificationTray.
This patch also adds a Windows port of MessageCenterTray -
MessageCenterTrayWin.
BUG=168605
TEST:
message_center_unittests
browser_tests (WebNotificationTrayWinTest.*)
ash_unittests (WebNotificationTrayTest.*)
Review URL: https://chromiumcodereview.appspot.com/11819048
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179807 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=124830
TEST=New tests
Review URL: https://codereview.chromium.org/12089078
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179743 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
An iterator meant to iterate over all the browsers in BrowserListImpls across multiple desktops (i.e. multiple HostDesktopTypes).
As a first step, used it in TabContentsIterator (pretty much bringing back the implementation to the one that was there prior to https://codereview.chromium.org/11316340/ -- except that the iterator is now aware of multiple desktops).
Another CL will follow in which all BrowserList::const_iterator, etc. methods will be replaced by BrowserIterator.
The interface of BrowserIterator is derived from the interface of TabContentsIterator for consistency (which is being slightly tweaked in parallel in https://codereview.chromium.org/12049038/).
BUG=129187
Review URL: https://chromiumcodereview.appspot.com/12038053
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179695 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Revert 179554 - suspected to break browser_tests on Mac 10.6
>
> > Refactor FullscreenController removing TogglePresentationMode & adding ToggleFullscreenWithChrome.
> >
> > Part of a series of refactoring changes that will enable simpler code in FullscreenController as well as correcting behavior there. This change attempts to make the minimal modifications required in order to have FullscreenController::ToggleFullscreenMode consistently mean fullscreen with no browser chrome on all platforms.
> >
> > Depends on https://codereview.chromium.org/12018007/
> >
> > BUG=169138
> >
> > Review URL: https://chromiumcodereview.appspot.com/11896104
>
> TBR=scheib@chromium.org
> Review URL: https://codereview.chromium.org/12089068
TBR=kinuko@chromium.org
Review URL: https://codereview.chromium.org/12096059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179626 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Refactor FullscreenController removing TogglePresentationMode & adding ToggleFullscreenWithChrome.
>
> Part of a series of refactoring changes that will enable simpler code in FullscreenController as well as correcting behavior there. This change attempts to make the minimal modifications required in order to have FullscreenController::ToggleFullscreenMode consistently mean fullscreen with no browser chrome on all platforms.
>
> Depends on https://codereview.chromium.org/12018007/
>
> BUG=169138
>
> Review URL: https://chromiumcodereview.appspot.com/11896104
TBR=scheib@chromium.org
Review URL: https://codereview.chromium.org/12089068
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179615 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
simple_message_box_win.cc is not be needed nor compatible with the desktop-aura
transition. It also uses the win32 appearance not the new Chrome style
appearance currently in development. So it's bad twice ;)
Just kill it and that eases the work that will need to be done later.
By enabling simple_message_box_views on Windows means that it will be used for all
ShowMessageBox() dialogs instead of current Windows MessageBox.
BUG=166075,54988
R=ben@chromium.org
Review URL: https://codereview.chromium.org/12042083
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179608 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is in preparation for adding chrome-app link handlers which
will also require this popup.
BUG=
Review URL: https://chromiumcodereview.appspot.com/12051033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179558 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ToggleFullscreenWithChrome.
Part of a series of refactoring changes that will enable simpler code in FullscreenController as well as correcting behavior there. This change attempts to make the minimal modifications required in order to have FullscreenController::ToggleFullscreenMode consistently mean fullscreen with no browser chrome on all platforms.
Depends on https://codereview.chromium.org/12018007/
BUG=169138
Review URL: https://chromiumcodereview.appspot.com/11896104
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179554 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implemented drop hander for the Home toolbar button that accepts links
and informs the user with a bubble with the option to undo.
Contributed by: tom.cassiotis@gmail.com
BUG=152210
TEST=Manually tested various links and other sources and followed
through all confirmation permutations (Undo, clicking on bubble, focus
loss/application switching, navigating off page). Also the special
case where the setting that the undo is resetting to is the NTP.
Tested RTL alignment to ensure that the Undo link is to the left
of the message in this situation.
Review URL: https://chromiumcodereview.appspot.com/11742003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179526 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This relands r179307 with the app_list_unittests target updated on chromeos.
TBR=sky@chromium.org, atwilson@chromium.org, rogerta@chromium.org, xiyuan@chromium.org
BUG=159733
Review URL: https://chromiumcodereview.appspot.com/12093040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179453 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
to content.
BUG=169170
Review URL: https://codereview.chromium.org/12089034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179425 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Revert 179271
> > Simplify how TestChromeWebUIControllerFactory is registered now that multiple WebUIControllerFactory instances are supported. Instead of compiling the test code into the shipping product, have tests which use it unregister the default one and register the test one. This also makes the non-test version used for the majority of the test, which keeps the browser_tests more like the shipping product.
> >
> > I've also moved the test harness code to chrome\test\base, which is where test harnesses live.
> >
> > BUG=169170
> > Review URL: https://codereview.chromium.org/12084029
>
> This seems cause a few webui test crashes on Linux ChromiumOS Tests
> (dbg)(2) since http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%282%29/builds/6808
>
> I can reproduce the crash on my local machine with CookiesViewWebUITest.*
> (Note: not all failing tests can be reproduced but this one has a reliable
> reproduce of the crash).
> And after I revert this, the crash gone. I am not sure why chromeos try
> bots have a green run.
>
> The crash is:
> [17690:17690:0128/215636:6223762366:FATAL:web_ui_impl.cc(84)] Check failed: false. Blocked unauthorized use of WebUIBindings.
> [0x7f4eb3da52f0] base::debug::StackTrace::StackTrace()
> [0x7f4eb3dda2a3] logging::LogMessage::~LogMessage()
> [0x7f4eaf52c45b] content::WebUIImpl::OnWebUISend()
> [0x7f4eaf52deaa] DispatchToMethod<>()
> [0x7f4eaf52d823] ViewHostMsg_WebUISend::Dispatch<>()
> [0x7f4eaf52c28c] content::WebUIImpl::OnMessageReceived()
> [0x7f4eaf4fe6e8] content::WebContentsImpl::OnMessageReceived()
> [0x7f4eaf444489] content::RenderViewHostImpl::OnMessageReceived()
> [0x7f4eaf42f031] content::RenderProcessHostImpl::OnMessageReceived()
> [0x7f4eb40109c0] IPC::ChannelProxy::Context::OnDispatchMessage()
> [0x7f4eb4014a8b] base::internal::RunnableAdapter<>::Run()
> [0x7f4eb4014553] base::internal::InvokeHelper<>::MakeItSo()
> [0x7f4eb4013df3] base::internal::Invoker<>::Run()
> [0x7f4eb3d9a0cd] base::Callback<>::Run()
> [0x7f4eb3de0d21] MessageLoop::RunTask()
> [0x7f4eb3de0e3c] MessageLoop::DeferOrRunPendingTask()
> [0x7f4eb3de16cd] MessageLoop::DoWork()
> [0x7f4eb3d7aee7] base::MessagePumpGlib::HandleDispatch()
> [0x7f4eb3d7a557] (anonymous namespace)::WorkSourceDispatch()
> [0x7f4ea3adb9d2] g_main_context_dispatch
> [0x7f4ea3adf858] <unknown>
> [0x7f4ea3adfa0c] g_main_context_iteration
> [0x7f4eb3d7aaf4] base::MessagePumpGlib::RunWithDispatcher()
> [0x7f4eb3d7afc4] base::MessagePumpGlib::Run()
> [0x7f4eb3de0911] MessageLoop::RunInternal()
> [0x7f4eb3de07cc] MessageLoop::RunHandler()
> [0x7f4eb3e197c6] base::RunLoop::Run()
> [0x000003101379] content::RunThisRunLoop()
> [0x000003101306] content::RunMessageLoop()
> [0x000001b66d64] InProcessBrowserTest::QuitBrowsers()
> [0x000001b66c58] InProcessBrowserTest::RunTestOnMainThreadLoop()
> [0x0000030bd6dd] content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
> [0x0000030bdb5b] base::internal::RunnableAdapter<>::Run()
> [0x0000030bdaf1] base::internal::InvokeHelper<>::MakeItSo()
> [0x0000030bdaa9] base::internal::Invoker<>::Run()
> [0x000000775215] base::Callback<>::Run()
> [0x00000333188c] ChromeBrowserMainParts::PreMainMessageLoopRunImpl()
> [0x00000332fe46] ChromeBrowserMainParts::PreMainMessageLoopRun()
> [0x000002c50658] chromeos::ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun()
> [0x7f4eaf197cba] content::BrowserMainLoop::CreateThreads()
> [0x7f4eaf19b57e] content::BrowserMainRunnerImpl::Initialize()
> [0x7f4eaf1959f7] content::BrowserMain()
> [0x0000030bd662] content::BrowserTestBase::SetUp()
> [0x000001b65e1a] InProcessBrowserTest::SetUp()
> [0x000001c8072b] testing::internal::HandleSehExceptionsInMethodIfSupported<>()
> [0x000001c7de76] testing::internal::HandleExceptionsInMethodIfSupported<>()
> [0x000001c7305d] testing::Test::Run()
> [0x000001c738d6] testing::TestInfo::Run()
> [0x000001c74004] testing::TestCase::Run()
> [0x000001c78faf] testing::internal::UnitTestImpl::RunAllTests()
> [0x000001c81557] testing::internal::HandleSehExceptionsInMethodIfSupported<>()
> [0x000001c7e67d] testing::internal::HandleExceptionsInMethodIfSupported<>()
> [0x000001c77a7e] testing::UnitTest::Run()
> [0x000001b8183c] base::TestSuite::Run()
> [0x000000e6b473] ChromeTestLauncherDelegate::RunTestSuite()
> [0x0000030fd3fc] content::LaunchTests()
> [0x000000e6b315] main
> [0x7f4ea2d7fc4d] __libc_start_main
> [0x00000072fc29] <unknown>
>
> TBR=jam@chromium.org
> Review URL: https://codereview.chromium.org/12089051
TBR=bshe@chromium.org
Review URL: https://codereview.chromium.org/12093047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179352 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Simplify how TestChromeWebUIControllerFactory is registered now that multiple WebUIControllerFactory instances are supported. Instead of compiling the test code into the shipping product, have tests which use it unregister the default one and register the test one. This also makes the non-test version used for the majority of the test, which keeps the browser_tests more like the shipping product.
>
> I've also moved the test harness code to chrome\test\base, which is where test harnesses live.
>
> BUG=169170
> Review URL: https://codereview.chromium.org/12084029
This seems cause a few webui test crashes on Linux ChromiumOS Tests
(dbg)(2) since http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%282%29/builds/6808
I can reproduce the crash on my local machine with CookiesViewWebUITest.*
(Note: not all failing tests can be reproduced but this one has a reliable
reproduce of the crash).
And after I revert this, the crash gone. I am not sure why chromeos try
bots have a green run.
The crash is:
[17690:17690:0128/215636:6223762366:FATAL:web_ui_impl.cc(84)] Check failed: false. Blocked unauthorized use of WebUIBindings.
[0x7f4eb3da52f0] base::debug::StackTrace::StackTrace()
[0x7f4eb3dda2a3] logging::LogMessage::~LogMessage()
[0x7f4eaf52c45b] content::WebUIImpl::OnWebUISend()
[0x7f4eaf52deaa] DispatchToMethod<>()
[0x7f4eaf52d823] ViewHostMsg_WebUISend::Dispatch<>()
[0x7f4eaf52c28c] content::WebUIImpl::OnMessageReceived()
[0x7f4eaf4fe6e8] content::WebContentsImpl::OnMessageReceived()
[0x7f4eaf444489] content::RenderViewHostImpl::OnMessageReceived()
[0x7f4eaf42f031] content::RenderProcessHostImpl::OnMessageReceived()
[0x7f4eb40109c0] IPC::ChannelProxy::Context::OnDispatchMessage()
[0x7f4eb4014a8b] base::internal::RunnableAdapter<>::Run()
[0x7f4eb4014553] base::internal::InvokeHelper<>::MakeItSo()
[0x7f4eb4013df3] base::internal::Invoker<>::Run()
[0x7f4eb3d9a0cd] base::Callback<>::Run()
[0x7f4eb3de0d21] MessageLoop::RunTask()
[0x7f4eb3de0e3c] MessageLoop::DeferOrRunPendingTask()
[0x7f4eb3de16cd] MessageLoop::DoWork()
[0x7f4eb3d7aee7] base::MessagePumpGlib::HandleDispatch()
[0x7f4eb3d7a557] (anonymous namespace)::WorkSourceDispatch()
[0x7f4ea3adb9d2] g_main_context_dispatch
[0x7f4ea3adf858] <unknown>
[0x7f4ea3adfa0c] g_main_context_iteration
[0x7f4eb3d7aaf4] base::MessagePumpGlib::RunWithDispatcher()
[0x7f4eb3d7afc4] base::MessagePumpGlib::Run()
[0x7f4eb3de0911] MessageLoop::RunInternal()
[0x7f4eb3de07cc] MessageLoop::RunHandler()
[0x7f4eb3e197c6] base::RunLoop::Run()
[0x000003101379] content::RunThisRunLoop()
[0x000003101306] content::RunMessageLoop()
[0x000001b66d64] InProcessBrowserTest::QuitBrowsers()
[0x000001b66c58] InProcessBrowserTest::RunTestOnMainThreadLoop()
[0x0000030bd6dd] content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
[0x0000030bdb5b] base::internal::RunnableAdapter<>::Run()
[0x0000030bdaf1] base::internal::InvokeHelper<>::MakeItSo()
[0x0000030bdaa9] base::internal::Invoker<>::Run()
[0x000000775215] base::Callback<>::Run()
[0x00000333188c] ChromeBrowserMainParts::PreMainMessageLoopRunImpl()
[0x00000332fe46] ChromeBrowserMainParts::PreMainMessageLoopRun()
[0x000002c50658] chromeos::ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun()
[0x7f4eaf197cba] content::BrowserMainLoop::CreateThreads()
[0x7f4eaf19b57e] content::BrowserMainRunnerImpl::Initialize()
[0x7f4eaf1959f7] content::BrowserMain()
[0x0000030bd662] content::BrowserTestBase::SetUp()
[0x000001b65e1a] InProcessBrowserTest::SetUp()
[0x000001c8072b] testing::internal::HandleSehExceptionsInMethodIfSupported<>()
[0x000001c7de76] testing::internal::HandleExceptionsInMethodIfSupported<>()
[0x000001c7305d] testing::Test::Run()
[0x000001c738d6] testing::TestInfo::Run()
[0x000001c74004] testing::TestCase::Run()
[0x000001c78faf] testing::internal::UnitTestImpl::RunAllTests()
[0x000001c81557] testing::internal::HandleSehExceptionsInMethodIfSupported<>()
[0x000001c7e67d] testing::internal::HandleExceptionsInMethodIfSupported<>()
[0x000001c77a7e] testing::UnitTest::Run()
[0x000001b8183c] base::TestSuite::Run()
[0x000000e6b473] ChromeTestLauncherDelegate::RunTestSuite()
[0x0000030fd3fc] content::LaunchTests()
[0x000000e6b315] main
[0x7f4ea2d7fc4d] __libc_start_main
[0x00000072fc29] <unknown>
TBR=jam@chromium.org
Review URL: https://codereview.chromium.org/12089051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179347 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
It's currently behind the --enable-memory-internals-ui command-line flag
BUG=168642
R=jhawkins@chromium.org,marja@chromium.org
Review URL: https://codereview.chromium.org/11896108
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179319 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Breaks linking on app_list_unittest on Linux: symbol lookup error: libcontent.so: undefined symbol: _ZN15MallocExtension32GetBytesAllocatedOnCurrentThreadEv
> [win] Add signin support for the app launcher.
>
> This relands r178795, with the navigation to the sign in screen deferred until
> the app list is shown and sign in is needed.
>
> TBR=sky@chromium.org, atwilson@chromium.org, rogerta@chromium.org, xiyuan@chromium.org
> BUG=159733
>
> Review URL: https://codereview.chromium.org/12095027
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/12088037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179307 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This relands r178795, with the navigation to the sign in screen deferred until
the app list is shown and sign in is needed.
TBR=sky@chromium.org, atwilson@chromium.org, rogerta@chromium.org, xiyuan@chromium.org
BUG=159733
Review URL: https://codereview.chromium.org/12095027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179302 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
webui pages.
BUG=169170
Review URL: https://codereview.chromium.org/12087040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179290 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
multiple WebUIControllerFactory instances are supported. Instead of compiling the test code into the shipping product, have tests which use it unregister the default one and register the test one. This also makes the non-test version used for the majority of the test, which keeps the browser_tests more like the shipping product.
I've also moved the test harness code to chrome\test\base, which is where test harnesses live.
BUG=169170
Review URL: https://codereview.chromium.org/12084029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179271 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=172013
Review URL: https://chromiumcodereview.appspot.com/12045081
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179027 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Here's how the change set makes it look like:
A glow on the bookmarks has also been added per suggestion of sail@.
http://i.imgur.com/oJ5Mv.png
http://i.imgur.com/1Y2XA.png
http://i.imgur.com/NtXeA.png
http://i.imgur.com/AMeFi.png
http://i.imgur.com/YOYnb.png
BUG=168858
TBR=thakis@chromium.org
Review URL: https://chromiumcodereview.appspot.com/11916004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@178983 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Now the new AppIconLoader is shared between launcher and message center settings.
BUG=161094
Review URL: https://chromiumcodereview.appspot.com/12033069
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@178910 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Speculative revert to see whether this is responsible for CaptivePortalBrowserTest failures.
Tests are failing because there are more navigations happening than expected.
> Added signin support for the app launcher.
>
> This prompts the user to sign in if the app launcher is shown and there is no authenticated user in the Chrome profile. This does not happen on ChromeOS.
>
> BUG=159733
>
>
> Review URL: https://chromiumcodereview.appspot.com/12044002
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/12052077
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@178816 0039d316-1c4b-4281-b951-d872f2087c98
|