- 17 Jul, 2014 40 commits
-
-
ttuttle@chromium.org authored
Add a bunch more baked-in configs, and update the existing ones. Also, improve bake_in_configs.py so it will display multiple errors before failing, and re-enable the unit test that failed when the existing configs expired. BUG= Review URL: https://codereview.chromium.org/391383003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283657 0039d316-1c4b-4281-b951-d872f2087c98
-
rdevlin.cronin@chromium.org authored
Allow a menu to stay open when a child View (not a MenuItemView) is doing a drag-and-drop operation. We need this in order to allow the wrench menu to stay open during browser action drag-and-drop when the extension action redesign switch is enabled. BUG=393038 Review URL: https://codereview.chromium.org/378333003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283656 0039d316-1c4b-4281-b951-d872f2087c98
-
mukai@chromium.org authored
It seems this file doesn't exist anymore, so the exclusion rules aren't also necessary. And GN reports error for excluding non-existing files. BUG=None R=brettw@chromium.org, szym@chromium.org TEST=None Review URL: https://codereview.chromium.org/399723002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283655 0039d316-1c4b-4281-b951-d872f2087c98
-
sergeyu@chromium.org authored
Moved all files from remoting/jingle_glue to remoting/signaling and remoting/protocol. The new remoting/signaling directory contains signaling layer implementation for remoting that can be used independently of remoting/protocol. TBR=rch@chromium.org Review URL: https://codereview.chromium.org/390983003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283654 0039d316-1c4b-4281-b951-d872f2087c98
-
suderman@chromium.org authored
BUG=47554 Review URL: https://codereview.chromium.org/394703004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283653 0039d316-1c4b-4281-b951-d872f2087c98
-
tsepez@chromium.org authored
This is breaking ipc fuzzer build. TBR=jam@chromium.org Review URL: https://codereview.chromium.org/391373005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283652 0039d316-1c4b-4281-b951-d872f2087c98
-
xhwang@chromium.org authored
Previously CdmAdapter only allows one outstanding SendPlatformChallenge() call. However, CDM actually needs to make parallel SendPlatformChallenge() calls. This CL fixes CdmAdapter to allow this. BUG=393722 TEST=Tested on ChromeOS build on Linux with hack in PepperPlatformVerificationMessageFilter to return faked challenge platform reply. Review URL: https://codereview.chromium.org/393713003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283651 0039d316-1c4b-4281-b951-d872f2087c98
-
erikchen@chromium.org authored
-----------------Reland 2 Description------------------ Turned on layers for the test for fullscreen windows. -----------------Reland 1 Description------------------ Reland 1 link: https://codereview.chromium.org/393933003/ Core animation was turned on in M35. The fullscreen window was not layer backed, but should have been. This original CL exposed this bug. -----------------Original Description------------------ Original CL link: https://codereview.chromium.org/379293003/ In OSX 10.10+, all views must be added to the NSWindow's contentView. Some views (like the tab strip and the profile icon) are placed on top of the title bar and require special treatment. All other views are added as subviews of 'chromeContentView' in TabWindowController. This allows tab dragging and fullscreen logic to easily move the views that don't need special treatment. This CL also removes the instances where a VersionIndependentWindow's contentView gets replaced by setContentView:. Instead, the 'chromeContentView' gets passed around as a subview. This allows VersionIndependentWindow to remove another of its internal hacks. BUG=392239 Review URL: https://codereview.chromium.org/399803002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283650 0039d316-1c4b-4281-b951-d872f2087c98
-
reed@google.com authored
will remove the flag from the skia side after this lands BUG= Review URL: https://codereview.chromium.org/396733009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283649 0039d316-1c4b-4281-b951-d872f2087c98
-
stevenjb@chromium.org authored
BUG=393266 For log only change to profile_io_data.cc (reviewed by mattm) TBR=mmenke@chromium.org Review URL: https://codereview.chromium.org/385993004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283648 0039d316-1c4b-4281-b951-d872f2087c98
-
kelvinp@chromium.org authored
This CL handles create a helper method in browser_tests to dismiss the host-out-of-date warning if it is displayed. BUG=393072 Review URL: https://codereview.chromium.org/389673003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283647 0039d316-1c4b-4281-b951-d872f2087c98
-
tfarina@chromium.org authored
build/grit_target.gypi does this for us, clients/consumers just need to define grit_out_dir variable and include grit_target.gypi for the magic happen. Found these entries with the following command line: $ for f in $(g ls-files | grep resources.gyp); do grep direct $f; echo $f; done $ for f in $(g ls-files | grep strings.gyp); do grep direct $f; echo $f; done BUG=None TEST=None TBR=keybuk@chromium.org,yoz@chromium.org,darin@chromium.org keybuk for device/bluetooth/ yoz for extensions/ darin for webkit/ Review URL: https://codereview.chromium.org/389223002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283646 0039d316-1c4b-4281-b951-d872f2087c98
-
petarj@mips.com authored
Fix the MIPS Linux build that was broken by r282266 (and r282246). Related r282266 review is at https://codereview.chromium.org/382743002 TEST= build Chrome Linux for target_arch=mipsel Review URL: https://codereview.chromium.org/396243002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283645 0039d316-1c4b-4281-b951-d872f2087c98
-
msw@chromium.org authored
r280376 changed the button drag image painting code. The label colors used make the text appear blurry. Use label native theme colors instead of button colors. (the black-on-white looks better than black-on-gray) See before/after pics at http://crbug.com/394245#c4 BUG=394245 TEST=Bookmark/URL drag images look okay. R=sky@chromium.org Review URL: https://codereview.chromium.org/400443002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283644 0039d316-1c4b-4281-b951-d872f2087c98
-
lliabraa@chromium.org authored
If no override is given, simHomePath needs to be initialized or iossim won't know where to find files (e.g. the simulator's sandboxed system.log) BUG=None Review URL: https://codereview.chromium.org/398133002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283643 0039d316-1c4b-4281-b951-d872f2087c98
-
davidben@chromium.org authored
BUG=none Review URL: https://codereview.chromium.org/395303003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283642 0039d316-1c4b-4281-b951-d872f2087c98
-
miletus@chromium.org authored
Currently we associate touchscreen to display based on matching resolution. But there could be cases the internal touchscreen reports a resolution that does not match any display resolution, or the internal display and external display have the same resolution so we don't know which display to match the touchscreen to. This patch adds a logic that if we have internal touchscreen and internal display, we should match them first regardless of the resolution. We already have the information about which display is internal display. For deciding internal touchscreen, we query X to get the touchscreen's /dev/input/eventX node, and call a script is_touchscreen_internal (which checks if the device is a I2C device) to decide whether the touchscreen is internal or not. BUG=chrome-os-partner:29398 TEST=touch position is correctly mapped on Blaze in extending mode, with external monitor having 1920x1280 resolution. Review URL: https://codereview.chromium.org/394063004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283641 0039d316-1c4b-4281-b951-d872f2087c98
-
andresantoso@chromium.org authored
With https://codereview.chromium.org/331993009/, GetMenu() may return NULL if running a native Cocoa menu. This patch deletes GetMenu and updates call sites to no longer call it. BUG=388455 Review URL: https://codereview.chromium.org/393943006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283640 0039d316-1c4b-4281-b951-d872f2087c98
-
tonyg@chromium.org authored
BUG=337829 Review URL: https://codereview.chromium.org/402433002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283639 0039d316-1c4b-4281-b951-d872f2087c98
-
noms@chromium.org authored
In the "delete the only active profile and create a new one" case, the newly created profile should use the placeholder avatar, and not one of the custom ones. The same should happen for the first run default profile. BUG=NONE TEST=Start Chrome with --new-profile-management. Delete all the profiles you have. The newly created profile should use the default guest-looking avatar (which you can check in the avatar bubble). Review URL: https://codereview.chromium.org/392313002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283635 0039d316-1c4b-4281-b951-d872f2087c98
-
pneubeck@chromium.org authored
KeygenHandler defaulted to use the persistent key slot. However, the Handler is also used in ChromeOS where no such global persistent slot exists. This change removes this default behavior from KeygenHandler. Instead only the slot provided by the CryptoModuleDelegate will be used. If no CryptoModuleDelegate is set, then an error will be thrown on keygen usage. The unit test of KeygenHandler now explicitly provides the slot through a StubCryptoModuleDelegate. In a follow up, the crypto::GetPersistentNSSKeySlot() call will be removed from the unit tests as well and instead the test slot will be get()'ed from the ScopedTestNSSDB. BUG=210525 Review URL: https://codereview.chromium.org/384413004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283634 0039d316-1c4b-4281-b951-d872f2087c98
-
pkotwicz@chromium.org authored
In particular, Widget::SetCapture() fails if the widget is not visible. BUG=None TEST=WidgetCaptureTest.FailedCaptureRequestIsNoop Review URL: https://codereview.chromium.org/375593002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283633 0039d316-1c4b-4281-b951-d872f2087c98
-
gunsch@chromium.org authored
Adapts to changes in https://codereview.chromium.org/381113004. R=lcwu@chromium.org,brettw@chromium.org BUG=None Review URL: https://codereview.chromium.org/396733002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283632 0039d316-1c4b-4281-b951-d872f2087c98
-
gunsch@chromium.org authored
BUG=None Review URL: https://codereview.chromium.org/395503005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283631 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283630 0039d316-1c4b-4281-b951-d872f2087c98
-
rlarocque@chromium.org authored
Rewrites the themes sync integration tests so they no longer depend on AwaitQuiescence() or related functions. Removes racy themes tests. They don't offer significantly better coverage than non-racy tests. Most of them were flaky and partially disabled. Adds or modifies existing tests so we have some coverage of both model association and ProcessSyncChanges(). BUG=97780,95742,84575,304554 Review URL: https://codereview.chromium.org/395503004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283629 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
8f75c67 Disable warning 4267 in pdfium until it is fixed. e76e823 Remove unused variable orig_size. 3d07b6f Do not disable warnings-as-errors for pdfium. BUG=375114 R=thestig@chromium.org TBR=thestig@chromium.org Review URL: https://codereview.chromium.org/400483002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283628 0039d316-1c4b-4281-b951-d872f2087c98
-
hiroshige@chromium.org authored
Create bridge functions that converts arguments into Blink objects (such as WebFileSystemEntry and WebFileInfo) after crossing threads and then call WebFileSystemCallbacks methods. BUG=393900 Review URL: https://codereview.chromium.org/391083002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283627 0039d316-1c4b-4281-b951-d872f2087c98
-
sashab@chromium.org authored
The permissions section in the App Info dialog originally started with 'It can'. Changed it to start with 'Permissions', so it is consistent with the other dialog headings. BUG=386871 Review URL: https://codereview.chromium.org/395653002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283626 0039d316-1c4b-4281-b951-d872f2087c98
-
dmikurube@chromium.org authored
BUG=382489 NOTRY=True Review URL: https://codereview.chromium.org/391643002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283625 0039d316-1c4b-4281-b951-d872f2087c98
-
groby@chromium.org authored
BUG=394624 R=thestig@chromium.org TBR=mmenke@chromium.org, thestig@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/399013002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283621 0039d316-1c4b-4281-b951-d872f2087c98
-
tapted@chromium.org authored
He's changing teams :'( NOTRY=true TBR=jochen@chromium.org Review URL: https://codereview.chromium.org/391293002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283620 0039d316-1c4b-4281-b951-d872f2087c98
-
estade@chromium.org authored
BUG=160586 Review URL: https://codereview.chromium.org/399433005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283619 0039d316-1c4b-4281-b951-d872f2087c98
-
zhaoqin@google.com authored
TBR=earthdok@chromium.org BUG=340752 Review URL: https://codereview.chromium.org/398053005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283618 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-release@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283615 0039d316-1c4b-4281-b951-d872f2087c98
-
viettrungluu@chromium.org authored
Blech: Stuff fails. Shockingly. http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%284%29/builds/27268/steps/mojo_shell_tests/logs/stdio > Mojo: Enable mojo_shell_tests on Windows (on the bots). > > R=sky@chromium.org > > Review URL: https://codereview.chromium.org/398023002 TBR=viettrungluu@chromium.org Review URL: https://codereview.chromium.org/398253002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283610 0039d316-1c4b-4281-b951-d872f2087c98
-
viettrungluu@chromium.org authored
ASan failure (use-after-free) looks like it's due to this change): http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Te... E.g.: @@@STEP_LOG_LINE@ReloadEmbedder@SUMMARY: AddressSanitizer: heap-use-after-free content/renderer/render_frame_proxy.cc:168 content::RenderFrameProxy::OnMessageReceived(IPC::Message const&)@@@j > Changes to RenderFrameProxy: > > - Add accessors: web_frame(), routing_id(), render_view(). > - Remove accessor: render_frame(). Where we do need to touch the > RenderFrame, we'll look it up by its routing ID. > - Small change to the CompositingHelper to use the new getters. > - Add a map to allow finding a RenderFrameProxy by its associated > blink::WebFrame. > - Introduce a second factory function and differentiate the two > factory functions according to the two ways RenderFrameProxies will > be created. The first is for when an extant local RenderFrame is > being swapped out and replaced with a new RenderFrameProxy. The > second is for when a RenderFrameProxy needs to be created without > displacing an existing RenderFrame, as shall occur once we mirror > the frame tree. > - This second factory function, which is uncalled at the moment, will > create WebRemoteFrames. Also there is stubbed out code in the first > factory function to create WebRemoteFrames. This code is in > preparation for eliminating the RenderFrame (and its attendant > WebLocalFrame) and having instead just a RenderFrameProxy. > - Add some defensive checks to prepare for when the render frame may > not exist, as will happen once the second factory function > enters use. > - Add an Init function so that code can be shared between the two > factory functions. > > As an adminstrative note, this patch is a chunk of nasko's > larger "use RenderFrameProxyHost" effor (issue 241223002) > > BUG=357747 > TEST=browsertests, http://csreis.github.io/tests/cross-site-iframe.html renders after going cross-site under --site-per-process > > Review URL: https://codereview.chromium.org/357043006 TBR=nick@chromium.org Review URL: https://codereview.chromium.org/397103006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283608 0039d316-1c4b-4281-b951-d872f2087c98
-
viettrungluu@chromium.org authored
I accidentally left this out from r283441. TBR=davemoore@chromium.org Review URL: https://codereview.chromium.org/394263003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283606 0039d316-1c4b-4281-b951-d872f2087c98
-
viettrungluu@chromium.org authored
Per review comment from https://codereview.chromium.org/398553002/. Note that I'm keeping the |pass| and deleting the redundant assignment (so now the |pass| is required). The alternative would be to nest the other |if|s inside an |if ... is not None|, but I find that harder to read. TBR=davemoore@chromium.org Review URL: https://codereview.chromium.org/396243006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283605 0039d316-1c4b-4281-b951-d872f2087c98
-
viettrungluu@chromium.org authored
R=sky@chromium.org Review URL: https://codereview.chromium.org/395243002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283601 0039d316-1c4b-4281-b951-d872f2087c98
-